Bug #4083

[SDKs] Add "import _strptime" to work around potential Python bug.

Added by Peter Amstutz almost 5 years ago. Updated almost 5 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
10/17/2014
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
0.5


Subtasks

Task #4252: Review 4083-import-strptimeResolvedPeter Amstutz

Task #4151: Figure out what uses strptime and fix themResolvedPeter Amstutz

Associated revisions

Revision d9014288 (diff)
Added by Peter Amstutz almost 5 years ago

4083: Searched for everything using strptime and added import _strptime where it was missing. closes #4083

Revision 207136fe (diff)
Added by Brett Smith over 3 years ago

4083: crunchstat-summary imports _strptime.

Refs #4083 for rationale.
Refs #8933 where this was reported.
See also d9014288.

History

#1 Updated by Peter Amstutz almost 5 years ago

  • Description updated (diff)

#2 Updated by Tom Clegg almost 5 years ago

  • Story points set to 0.5

#3 Updated by Tom Clegg almost 5 years ago

  • Target version changed from Bug Triage to Arvados Future Sprints

#4 Updated by Ward Vandewege almost 5 years ago

  • Target version changed from Arvados Future Sprints to 2014-10-29 sprint

#5 Updated by Peter Amstutz almost 5 years ago

  • Assigned To set to Peter Amstutz

#6 Updated by Brett Smith almost 5 years ago

It's not clear that this is strictly necessary. The issue will only arise if you start threads that are the first thing to run strptime. If your threads don't run strptime, or your first thread runs it before launching other threads, there's no risk.

At the same time, doing the import anyway is low-risk. But at least look at the situation before making a change.

#7 Updated by Brett Smith almost 5 years ago

c5d5029 is good, thanks.

#8 Updated by Anonymous almost 5 years ago

  • Status changed from New to Resolved

Applied in changeset arvados|commit:d9014288ed04b79bde6f626636a931b449323287.

Also available in: Atom PDF