----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54699/ -----------------------------------------------------------
Review request for Sqoop, Attila Szabo and Szabolcs Vasas. 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 ----- src/java/org/apache/sqoop/mapreduce/OracleUpsertOutputFormat.java 88754e7 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