Re: [infinispan-dev] Transactions and using synchronizations instead of XA - and ISPN-1284

After spending some hours on ISPN-1284, I think we have a few conceptual problems with using Synchronizations. Here is the topic branch containing my work: https://github.com/maniksurtani/infinispan/tree/t_1284 So far what I have done is: * Changed defaults * Changed config validations to emit a warning and disable synchronisations if recovery is enabled * Update some tests that rely on XA to specifically set synchronisations to false But there are still some issues, and what specifically worry me is behaviour demonstrated by these two test failures: testModsCommit(org.infinispan.lock.StaleEagerLocksOnPrepareFailureTest) testNoModsCommit(org.infinispan.lock.StaleEagerLocksOnPrepareFailureTest) They both attempt to abort a transaction by throwing an exception during prepare. Now since we use Synchronizations by default, these failures do *not* abort the transaction since it is only seen by the SynchronizationAdapter in afterCommit(). Where does this stand, conceptually? With optimistic transactions, locks may not be able to be acquired at prepare-time. These transactions should fail.