You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sqoop.apache.org by ve...@apache.org on 2013/12/05 20:07:45 UTC

git commit: SQOOP-1250: Oracle connector is not disabling autoCommit on created connections

Updated Branches:
  refs/heads/trunk e98075976 -> 8d3454dc6


SQOOP-1250:  Oracle connector is not disabling autoCommit on created connections

(Jarek Jarcec Cecho via Venkat Ranganathan)


Project: http://git-wip-us.apache.org/repos/asf/sqoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/sqoop/commit/8d3454dc
Tree: http://git-wip-us.apache.org/repos/asf/sqoop/tree/8d3454dc
Diff: http://git-wip-us.apache.org/repos/asf/sqoop/diff/8d3454dc

Branch: refs/heads/trunk
Commit: 8d3454dc60661162c1daa510c52fd60c483e3d16
Parents: e980759
Author: Venkat Ranganathan <ve...@hortonworks.com>
Authored: Thu Dec 5 11:07:30 2013 -0800
Committer: Venkat Ranganathan <ve...@hortonworks.com>
Committed: Thu Dec 5 11:07:30 2013 -0800

----------------------------------------------------------------------
 src/java/org/apache/sqoop/manager/OracleManager.java | 6 ++++++
 1 file changed, 6 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/sqoop/blob/8d3454dc/src/java/org/apache/sqoop/manager/OracleManager.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/sqoop/manager/OracleManager.java b/src/java/org/apache/sqoop/manager/OracleManager.java
index 8008831..77b8ba3 100644
--- a/src/java/org/apache/sqoop/manager/OracleManager.java
+++ b/src/java/org/apache/sqoop/manager/OracleManager.java
@@ -331,6 +331,12 @@ public class OracleManager
     // Setting session time zone
     setSessionTimeZone(connection);
 
+    // Rest of the Sqoop code expects that the connection will have be running
+    // without autoCommit, so we need to explicitly set it to false. This is
+    // usually done directly by SqlManager in the makeConnection method, but
+    // since we are overriding it, we have to do it ourselves.
+    connection.setAutoCommit(false);
+
     return connection;
   }