sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Venkat Ranganathan" <n....@live.com>
Subject Re: Review Request 17650: SQOOP-1278: Allow use of uncommitted isolation on databases that support is an import option
Date Tue, 04 Feb 2014 18:22:42 GMT

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

(Updated Feb. 4, 2014, 6:22 p.m.)


Review request for Sqoop.


Changes
-------

Thanks for a quick review Jarcec.   Incorporated all your suggestions and fixes.

Thanks

Venkat


Bugs: SQOOP-1278
    https://issues.apache.org/jira/browse/SQOOP-1278


Repository: sqoop-trunk


Description
-------

Allow the use of read uncommitted isolation on all databases that support it.  Oracle does
not read uncommitted as a settable option in the JDBC drivers so it will behave as before
on Oracle.


Diffs (updated)
-----

  src/docs/user/common-args.txt 8a017f4 
  src/docs/user/import.txt 0db6d97 
  src/java/org/apache/sqoop/SqoopOptions.java 46e158c 
  src/java/org/apache/sqoop/mapreduce/DataDrivenImportJob.java b21560e 
  src/java/org/apache/sqoop/mapreduce/db/DBConfiguration.java be942ce 
  src/java/org/apache/sqoop/mapreduce/db/DBInputFormat.java 73ed94e 
  src/java/org/apache/sqoop/tool/BaseSqoopTool.java 6d6f1ea 
  src/test/com/cloudera/sqoop/TestSqoopOptions.java 686d398 

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


Testing
-------

One new test to test the option.  Explicitly tested with Oracle, DB2 and Postgresql.   All
unit tests passed


Thanks,

Venkat Ranganathan


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message