On 14 May 2009, at 09:57, Mircea Markus wrote:
Hi,
Currently the interface of these classes use a javax.transaction.Transaction parameter:
void prepare(List<? extends Modification> modifications, Transaction tx, boolean isOnePhase) throws CacheLoaderException;
The problem with this is that Transaction will not exist if this is a remotely originated call (we only propagate the GlobalTransaction on the remote nodes).
I've took a look at the CacheStore impls, they do not use any specific javax.transaction.Transaction logic/methods, but rather use the param as a key in their internal caches.
Do you see any issues with changing the signature of CacheStore to replace Transaction param with GlobalTransaction?