Re: [ZODB-Dev] zc.demostorage2 assumes too much?

2009-12-15 Thread Jim Fulton
On Thu, Nov 26, 2009 at 7:33 AM, Wolfgang Schnerring wrote: > Hello, > > zc.demostorage2 assumes that the tpc_begin() of its underlying > storages takes three parameters (transaction, tid, status), while both > MappingStorage and IStorage only know about the first two. > > This strikes me as risky

[ZODB-Dev] zc.demostorage2 assumes too much?

2009-11-26 Thread Wolfgang Schnerring
Hello, zc.demostorage2 assumes that the tpc_begin() of its underlying storages takes three parameters (transaction, tid, status), while both MappingStorage and IStorage only know about the first two. This strikes me as risky, and it does indeed break, for example when trying to use a DemoStorage