You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sqoop.apache.org by Filippo Causa <fi...@unicredit.eu> on 2015/08/17 11:48:59 UTC
Re: Review Request 36729: Patch for jira 1096

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36729/
-----------------------------------------------------------

(Updated Aug. 17, 2015, 9:48 a.m.)


Review request for Sqoop and Jarek Cecho.


Repository: sqoop-trunk


Description (updated)
-------

propagated  relaxed-isolation parameter setting to connection used for calculating max incremental value and split range. Use --relaxed-isoltaion inbstead of "WITH UR" clause.


Diffs
-----

  src/java/com/cloudera/sqoop/mapreduce/db/DBConfiguration.java 89f2b4f 
  src/java/com/cloudera/sqoop/tool/BaseSqoopTool.java a5f72f7 
  src/java/org/apache/sqoop/SqoopOptions.java 9405605 
  src/java/org/apache/sqoop/hive/TableDefWriter.java c9962e9 
  src/java/org/apache/sqoop/manager/MySQLManager.java e1d5a36 
  src/java/org/apache/sqoop/manager/OracleManager.java 69b613f 
  src/java/org/apache/sqoop/mapreduce/DataDrivenImportJob.java 260bc29 
  src/java/org/apache/sqoop/mapreduce/JdbcExportJob.java 93d438a 
  src/java/org/apache/sqoop/mapreduce/db/DBConfiguration.java a9b7e42 
  src/java/org/apache/sqoop/mapreduce/db/DBInputFormat.java 3a8e5d0 
  src/java/org/apache/sqoop/mapreduce/db/DBRecordReader.java a78eb06 
  src/java/org/apache/sqoop/mapreduce/db/DataDrivenDBInputFormat.java db96e41 
  src/java/org/apache/sqoop/mapreduce/db/DataDrivenDBRecordReader.java b734e05 
  src/java/org/apache/sqoop/mapreduce/db/OracleDBRecordReader.java 9058a55 
  src/java/org/apache/sqoop/mapreduce/hcat/SqoopHCatUtilities.java 4070c24 
  src/java/org/apache/sqoop/mapreduce/sqlserver/SqlServerRecordReader.java bc101c5 
  src/java/org/apache/sqoop/orm/ClassWriter.java 1c6f7f4 
  src/java/org/apache/sqoop/tool/BaseSqoopTool.java 4e2e66d 
  src/java/org/apache/sqoop/tool/ImportTool.java 39af42c 

Diff: https://reviews.apache.org/r/36729/diff/


Testing (updated)
-------

Test done with jdbc type 4


File Attachments (updated)
----------------

Transaction-Isolation Patch
  https://reviews.apache.org/media/uploaded/files/2015/08/13/9ebd4663-592d-448f-a888-94ee00ca84de__1096.patch


Thanks,

Filippo Causa