"Locking a single remote node" in 4.2.0ALPHA1

[changgeng]

I think before accquiring a remote lock, the current implementation will still try to acquire a local lock, is that correct?

Could we have an option to entirely disable the local lock? This will be very helpful to avoid deadlock(local vs. remote) when there's a heavy concurrent write load on a specific key.

Also I create a testcase with the attached cluster.xml, and it seems there's some bug that I got the following exception.

Caused by: java.lang.RuntimeException: Failure to marshal argument(s)
        at org.infinispan.remoting.transport.jgroups.CommandAwareRpcDispatcher$ReplicationTask.marshallCall(CommandAwareRpcDispatcher.java:242)
        at org.infinispan.remoting.transport.jgroups.CommandAwareRpcDispatcher$ReplicationTask.call(CommandAwareRpcDispatcher.java:266)
        at org.infinispan.remoting.transport.jgroups.CommandAwareRpcDispatcher.invokeRemoteCommands(CommandAwareRpcDispatcher.java:120)
        ... 42 more
Caused by: org.infinispan.marshall.NotSerializableException: org.infinispan.transaction.xa.DistDldGlobalTransaction
Caused by: an exception which occurred:
        in object org.infinispan.transaction.xa.DistDldGlobalTransaction@19a1027a
        in object org.infinispan.commands.tx.RollbackCommand@19a1027a

[an1310]

This is being tracked by https://jira.jboss.org/browse/ISPN-633

[mircea.markus]

I think before accquiring a remote lock, the current implementation will still try to acquire a local lock, is that correct?

Yes, a local lock is still acquired.

Could we have an option to entirely disable the local lock? This will be very helpful to avoid deadlock(local vs. remote) when there's a heavy concurrent write load on a specific key.

I think deadlock detection should be able to determine such a deadlock. Also if you the keyAffinityService to enforce key's locality you won't hit this problem (this is described here: http://community.jboss.org/wiki/LockingandConcurrency#Locking_a_single_remote_node) . On the other hand I think this is a interesting idea and I see utility for it. Any more comments on this?

[changgeng]

KeyAffinityService looks nice, but I'm not sure we can use it now because currently the key we are using is generated by database. Deadlock dection can help, yet it will be better if we could avoid the deadlock.