You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by xi...@apache.org on 2010/11/25 06:52:43 UTC

svn commit: r1038927 - /geronimo/daytrader/branches/2.1/modules/ejb/src/main/java/org/apache/geronimo/samples/daytrader/direct/TradeDirect.java

Author: xiaming
Date: Thu Nov 25 05:52:43 2010
New Revision: 1038927

URL: http://svn.apache.org/viewvc?rev=1038927&view=rev
Log:
DAYTRADER-83 fix pingservlet2session2jdbc failure

Modified:
    geronimo/daytrader/branches/2.1/modules/ejb/src/main/java/org/apache/geronimo/samples/daytrader/direct/TradeDirect.java

Modified: geronimo/daytrader/branches/2.1/modules/ejb/src/main/java/org/apache/geronimo/samples/daytrader/direct/TradeDirect.java
URL: http://svn.apache.org/viewvc/geronimo/daytrader/branches/2.1/modules/ejb/src/main/java/org/apache/geronimo/samples/daytrader/direct/TradeDirect.java?rev=1038927&r1=1038926&r2=1038927&view=diff
==============================================================================
--- geronimo/daytrader/branches/2.1/modules/ejb/src/main/java/org/apache/geronimo/samples/daytrader/direct/TradeDirect.java (original)
+++ geronimo/daytrader/branches/2.1/modules/ejb/src/main/java/org/apache/geronimo/samples/daytrader/direct/TradeDirect.java Thu Nov 25 05:52:43 2010
@@ -2020,7 +2020,9 @@ public class TradeDirect implements Trad
 		if (datasource == null)
 			getDataSource();
 		conn = datasource.getConnection();
-		conn.setAutoCommit(false);
+		if (!this.inSession) {
+			conn.setAutoCommit(false);
+		}
 		if (Log.doTrace()) {
 			synchronized (lock) {
 				connCount++;