On Mon, Dec 13, 2010 at 4:11 AM, Vincent Pelletier <vinc...@nexedi.com> wrote:
> Reading ZODB.interface, I realised the "version" parameter is defined as
> class IStorage(Interface):
> def load(oid, version):
> But some tests do not pass this parameter:
> $ grep "\.load([^,]*)" *.py
> BasicStorage.py: tid3 = self._storage.load(oid)
> BasicStorage.py: self.assert_(tid3 >
> BasicStorage.py: tid4 = self._storage.load(oid)
> BasicStorage.py: self.assert_(tid4 >
> I didn't review other matches, and my quick regex ignores word-wrapped lines.
> I think either should be fixed, but I'm not sure which (I tend to think
> interface should be fixed).
We should probably change the interface. I didn't want to do that
because that would require changes to existing storages.
For better or worse, I made another required storage change
lately to add the checkCurrentSerialInTransaction method, and I
should have cleaned these up at the same time.
For more information about ZODB, see the ZODB Wiki:
ZODB-Dev mailing list - ZODB-Dev@zope.org