----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54699/ -----------------------------------------------------------
(Updated Dec. 14, 2016, 9:52 a.m.) Review request for Sqoop, Attila Szabo and Szabolcs Vasas. Changes ------- made it compliant with 3066 + fixed per reviews Bugs: SQOOP-3081 https://issues.apache.org/jira/browse/SQOOP-3081 Repository: sqoop-trunk Description ------- SQOOP-3081 Introduce OracleEscapeUtils.escapeIdentifiers In OracleUpsertOutputFormat to make it consistent with the rest of the code and add escaping to the rest of the column variables. Diffs (updated) ----- src/java/org/apache/sqoop/SqoopOptions.java 53cc03a src/java/org/apache/sqoop/manager/oracle/OracleUtils.java aa56e70 src/java/org/apache/sqoop/mapreduce/OracleUpsertOutputFormat.java 88754e7 src/test/com/cloudera/sqoop/manager/OracleExportTest.java 382eb92 Diff: https://reviews.apache.org/r/54699/diff/ Testing ------- ran ant clean test, ant clean test -Dsqoop.thirdparty.lib.dir=$3rdpartylib -Dthirdparty=true -Dsqoop.test.oracle.connectstring=$connstr-Dmapred.child.java.opts="-Djava.security.egd=file:/dev/../dev/urandom" -Dtest.timeout=1000000 -Dtestcase=*racle*Test with special attention to OracleExportTest#testUpsertTextExport Thanks, Anna Szonyi