[infinispan-dev] Keeping track of locked nodes

[Mircea Markus]

On 20 Jan 2012, at 09:11, Dan Berindei wrote:
> In this particular case it just means that the commit is sync even if
> it's configured to be async.
> 
> But there are more places where we check the remote lock nodes list,
> e.g. BaseRpcInterceptor.shouldInvokeRemotely or
> AbstractEnlistmentAdapter - which, incidentally, could probably settle
> with a "hasRemoteLocks" boolean flag instead of the nodes collection.
> 
> TransactionXaAdapter.forgetSuccessfullyCompletedTransaction does use
> it properly when recovery is enabled - if we didn't keep the
> collection around it would have to compute it again from the list of
> keys.
> 
> The same with PessimisticLockingInterceptor.releaseLocksOnFailureBeforePrepare,
> but there we're on thefailure path already so recomputing the address
> set wouldn't hurt that much.
> 
> I may have missed others...
As mentioned, these can be determined based on the set of locks held by that tramsaction.