I'm fine with change the storage server to pass the history size
argument as a keyword parameter.

Jim

On Mon, Nov 2, 2009 at 6:04 AM, Chris Withers <ch...@simplistix.co.uk> wrote:
> Christian Theune wrote:
>>> Would anyone object if I wrote a test and fixed this on the 3.9 branch
>>> of ZODB? I'm not really sure what to do about the trunk...
>>
>> I think that's the best idea. It's not exactly part of the policy of
>> supporting old protocols but if we want to support and encourage 3rd
>> party storage implementations we should leverage this simple fix for
>> better backwards compatibility.
>
> Hmmm, I'm less sure now. The problematic code is here:
>
> http://zope3.pov.lt/trac/browser/ZODB/branches/3.9/src/ZEO/StorageServer.py#L1379
>
> It looks like ZEOStorage308Adapter is used when the protocol specified
> is less than Z309, which it is in my case. But, the assumption here is
> that the storage being served by StorageServer effectively matches the
> Z309 API. zeoraid's RAIDStorage does not...
>
> However, I can't see what `history` method is called if the protocol >=
> Z309. Christian, I'm guessing you guys have used zeoraid with 3.9 from
> some time now, does this issue not occur if everything is talking Z309?
> If not, why not?
>
> Chris
>
> --
> Simplistix - Content Management, Batch Processing & Python Consulting
>            - http://www.simplistix.co.uk
> _______________________________________________
> For more information about ZODB, see the ZODB Wiki:
> http://www.zope.org/Wikis/ZODB/
>
> ZODB-Dev mailing list  -  zodb-...@zope.org
> https://mail.zope.org/mailman/listinfo/zodb-dev
>



-- 
Jim Fulton
_______________________________________________
For more information about ZODB, see the ZODB Wiki:
http://www.zope.org/Wikis/ZODB/

ZODB-Dev mailing list  -  ZODB-Dev@zope.org
https://mail.zope.org/mailman/listinfo/zodb-dev

Reply via email to