Re: [Zope-dev] ZEO client cache not working properly with beforestorage
On Mon, Nov 7, 2011 at 3:41 AM, Malthe Borch wrote: > On 25 October 2011 17:53, Malthe Borch wrote: >> I'll try and see if I can understand what might be the right thing to >> expect and write a test case for that. > > Following up here on the list to get some feedback on the patch I > submitted concerning this issue: > > https://bugs.launchpad.net/zodb/+bug/881493 > > The patch includes a test case and necessary code modifications. > > If someone could review the test case included in the patch, I'd appreciate > it. Thanks. I'll review it. Not sure when :), but I'll definitely put this on my to-do list for 3.11. Jim -- Jim Fulton http://www.linkedin.com/in/jimfulton ___ Zope-Dev maillist - Zope-Dev@zope.org https://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - https://mail.zope.org/mailman/listinfo/zope-announce https://mail.zope.org/mailman/listinfo/zope )
Re: [Zope-dev] ZEO client cache not working properly with beforestorage
On 25 October 2011 17:53, Malthe Borch wrote: > I'll try and see if I can understand what might be the right thing to > expect and write a test case for that. Following up here on the list to get some feedback on the patch I submitted concerning this issue: https://bugs.launchpad.net/zodb/+bug/881493 The patch includes a test case and necessary code modifications. If someone could review the test case included in the patch, I'd appreciate it. \malthe ___ Zope-Dev maillist - Zope-Dev@zope.org https://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - https://mail.zope.org/mailman/listinfo/zope-announce https://mail.zope.org/mailman/listinfo/zope )
Re: [Zope-dev] ZEO client cache not working properly with beforestorage
On 25 October 2011 16:50, Jim Fulton wrote: > Would you mind creating an issue: > > https://bugs.launchpad.net/zodb/+filebug Definitely. Filed at: https://bugs.launchpad.net/zodb/+bug/881493 > Of course, patches including tests are appreciated. I'll try and see if I can understand what might be the right thing to expect and write a test case for that. Thanks, \malthe ___ Zope-Dev maillist - Zope-Dev@zope.org https://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - https://mail.zope.org/mailman/listinfo/zope-announce https://mail.zope.org/mailman/listinfo/zope )
Re: [Zope-dev] ZEO client cache not working properly with beforestorage
I take issue with "not working properly". Before storage works correctly with ZEO. On Tue, Oct 25, 2011 at 5:43 AM, Malthe Borch wrote: > If I set my beforestorage to a reasonably current time, e.g. "now", I > expect to get reasonably good performance from a ZEO client cache. > > Instead, I find that beforestorage: > > 1) asks the cache to ``loadBefore``; This is absolutely the right thing to do. > 2) but that method only looks for "noncurrent" entries; > 3) meanwhile, my cache only has "current" records. > > I'm not sure if the problem is that the cache shouldn't be setting the > loaded oids as "current", or if the ``loadBefore`` method should also > accept (some) "current" entries. I agree that the ZEO cache implementation should check for current records with tids < the before time. This has potential implications beyond beforestorage. Would you mind creating an issue: https://bugs.launchpad.net/zodb/+filebug Of course, patches including tests are appreciated. Jim -- Jim Fulton http://www.linkedin.com/in/jimfulton ___ Zope-Dev maillist - Zope-Dev@zope.org https://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - https://mail.zope.org/mailman/listinfo/zope-announce https://mail.zope.org/mailman/listinfo/zope )
[Zope-dev] ZEO client cache not working properly with beforestorage
If I set my beforestorage to a reasonably current time, e.g. "now", I expect to get reasonably good performance from a ZEO client cache. Instead, I find that beforestorage: 1) asks the cache to ``loadBefore``; 2) but that method only looks for "noncurrent" entries; 3) meanwhile, my cache only has "current" records. I'm not sure if the problem is that the cache shouldn't be setting the loaded oids as "current", or if the ``loadBefore`` method should also accept (some) "current" entries. \malthe ___ Zope-Dev maillist - Zope-Dev@zope.org https://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - https://mail.zope.org/mailman/listinfo/zope-announce https://mail.zope.org/mailman/listinfo/zope )