Comment 5 for bug 279945

Revision history for this message
James Henstridge (jamesh) wrote :

A test and some documentation would be useful. As far as the API goes, it'd probably be better to provide a "manage per thread stores" argument to the constructor rather than "factory for the _local attribute".

Even better would be if we could tell whether the ZStorm should be managing per-thread stores from transaction manager itself. I wonder if checking whether it is an instance of ThreadedTransactionManager is enough, or if there might be other subclasses where per-thread stores would be appropriate?