Author sbt
Recipients asksol, jnoller, meador.inge, sbt, vilnis.termanis
Date 2011-08-29.16:36:40
SpamBayes Score 6.17917e-11
Marked as misclassified No
Message-id <1314635801.15.0.00705068228588.issue8037@psf.upfronthosting.co.za>
In-reply-to
Content
Modifying an object which is already on a traditional queue can also change what is received by the other thread (depending on timing).  So Queue.Queue's put() is not "atomic" either.  Therefore I do not believe this behaviour is a bug. 

However the solution proposed is a good one since it fixes Issue 10886.  In addition it prevents arbitrary code being run in the background thread by weakref callbacks or __del__ methods.  Such arbitrary code may cause inconsistent state in a forked process if the fork happens while the queue's thread is running -- see issue 6271.

I have submitted a patch for Issue 10886.  It is basically the same as 
patch_27maint.diff, but it is against the default mercurial branch.  (Also, it is a bit simpler because does it does not unnecessarily modify Queue.get().)

I would suggest closing this issue and letting Issue 10886 take it's place.
History
Date User Action Args
2011-08-29 16:36:41sbtsetrecipients: + sbt, jnoller, asksol, meador.inge, vilnis.termanis
2011-08-29 16:36:41sbtsetmessageid: <1314635801.15.0.00705068228588.issue8037@psf.upfronthosting.co.za>
2011-08-29 16:36:40sbtlinkissue8037 messages
2011-08-29 16:36:40sbtcreate