This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author alexandre.vassalotti
Recipients Arfrever, alexandre.vassalotti, asvetlov, mstefanro, neologix, pitrou, rhettinger, serhiy.storchaka
Date 2013-05-12.00:51:06
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1368319880.77.0.0759989594967.issue17810@psf.upfronthosting.co.za>
In-reply-to
Content
Thanks Stefan for the patch. It's very much appreciated. I will try to review it soon.

Of the features you proposed, the twos I would like to take a look again is implicit memoization and the BAIL_OUT opcode. For the implicit memoization feature, we will need to have some performance results in hand to justify the major changes it needs. If you can you work out a quick patch, I can run it through the benchmarks suite for pickle and measure the impact. Hopefully, we will see a good improvement though we can't be sure until we measure.

And as for the BAIL_OUT opcode, it would be interesting to revisit its use now that we support binary framing. It could be helpful to add it to prevent the Unpickler from hanging if the other end forgot to close the stream. I am still not totally convinced. However if you make a good case for it, I would support to see it included.
History
Date User Action Args
2013-05-12 00:51:21alexandre.vassalottisetrecipients: + alexandre.vassalotti, rhettinger, pitrou, Arfrever, asvetlov, neologix, serhiy.storchaka, mstefanro
2013-05-12 00:51:20alexandre.vassalottisetmessageid: <1368319880.77.0.0759989594967.issue17810@psf.upfronthosting.co.za>
2013-05-12 00:51:20alexandre.vassalottilinkissue17810 messages
2013-05-12 00:51:06alexandre.vassalotticreate