[jboss-cvs] JBossAS SVN: r76313 - in trunk/connector/src/main/org/jboss/resource/adapter: jms and 1 other directory.
jboss-cvs-commits at lists.jboss.org
jboss-cvs-commits at lists.jboss.org
Mon Jul 28 13:02:26 EDT 2008
Author: adrian at jboss.org
Date: 2008-07-28 13:02:26 -0400 (Mon, 28 Jul 2008)
New Revision: 76313
Modified:
trunk/connector/src/main/org/jboss/resource/adapter/jdbc/xa/XAManagedConnection.java
trunk/connector/src/main/org/jboss/resource/adapter/jms/JmsXAResource.java
Log:
[JBAS-5801] - No need to lock during 2PC protocol, which interferes with interleaving. Locking durning end() is enough
Modified: trunk/connector/src/main/org/jboss/resource/adapter/jdbc/xa/XAManagedConnection.java
===================================================================
--- trunk/connector/src/main/org/jboss/resource/adapter/jdbc/xa/XAManagedConnection.java 2008-07-28 16:59:03 UTC (rev 76312)
+++ trunk/connector/src/main/org/jboss/resource/adapter/jdbc/xa/XAManagedConnection.java 2008-07-28 17:02:26 UTC (rev 76313)
@@ -271,41 +271,17 @@
public int prepare(Xid xid) throws XAException
{
- lock();
- try
- {
- return xaResource.prepare(xid);
- }
- finally
- {
- unlock();
- }
+ return xaResource.prepare(xid);
}
public void commit(Xid xid, boolean onePhase) throws XAException
{
- lock();
- try
- {
- xaResource.commit(xid, onePhase);
- }
- finally
- {
- unlock();
- }
+ xaResource.commit(xid, onePhase);
}
public void rollback(Xid xid) throws XAException
{
- lock();
- try
- {
- xaResource.rollback(xid);
- }
- finally
- {
- unlock();
- }
+ xaResource.rollback(xid);
}
public void forget(Xid xid) throws XAException
Modified: trunk/connector/src/main/org/jboss/resource/adapter/jms/JmsXAResource.java
===================================================================
--- trunk/connector/src/main/org/jboss/resource/adapter/jms/JmsXAResource.java 2008-07-28 16:59:03 UTC (rev 76312)
+++ trunk/connector/src/main/org/jboss/resource/adapter/jms/JmsXAResource.java 2008-07-28 17:02:26 UTC (rev 76313)
@@ -79,41 +79,17 @@
public int prepare(Xid xid) throws XAException
{
- managedConnection.lock();
- try
- {
- return xaResource.prepare(xid);
- }
- finally
- {
- managedConnection.unlock();
- }
+ return xaResource.prepare(xid);
}
public void commit(Xid xid, boolean onePhase) throws XAException
{
- managedConnection.lock();
- try
- {
- xaResource.commit(xid, onePhase);
- }
- finally
- {
- managedConnection.unlock();
- }
+ xaResource.commit(xid, onePhase);
}
public void rollback(Xid xid) throws XAException
{
- managedConnection.lock();
- try
- {
- xaResource.rollback(xid);
- }
- finally
- {
- managedConnection.unlock();
- }
+ xaResource.rollback(xid);
}
public void forget(Xid xid) throws XAException
More information about the jboss-cvs-commits
mailing list