[Patches] [ python-Patches-1690578 ] Don't block on Queue get/put when time is moved back

SourceForge.net noreply at sourceforge.net
Mon Apr 9 22:51:03 CEST 2007


Patches item #1690578, was opened at 2007-03-29 07:32
Message generated for change (Comment added) made by rhettinger
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1690578&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Library (Lib)
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: xiaowen (xiaowen)
Assigned to: Nobody/Anonymous (nobody)
Summary: Don't block on Queue get/put when time is moved back

Initial Comment:
When Queue get and put methods are called with a timeout, they effectively store the end time and might block until that end time is reached.  This breaks if the system time is moved back.  The attached patch breaks out of the wait if it detects the time has been moved back.

----------------------------------------------------------------------

>Comment By: Raymond Hettinger (rhettinger)
Date: 2007-04-09 15:51

Message:
Logged In: YES 
user_id=80475
Originator: NO

When a system moves time backwards, it breaks a host of invariants, not
just the one in the queue module.  I would not want some variant of this
code used everywhere time differences are computed.  Another thought is
that the use case is somewhat exceptional and that the solution
(readjusting endtime) isn't necessarily the right thing to do.

----------------------------------------------------------------------

Comment By: Josiah Carlson (josiahcarlson)
Date: 2007-04-09 15:36

Message:
Logged In: YES 
user_id=341410
Originator: NO

I am unsure if this is a sufficient or correct patch, but the
functionality is desirable.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1690578&group_id=5470


More information about the Patches mailing list