> On Dec. 13, 2016, 2:37 p.m., Erzsebet Szilagyi wrote: > > Thank you! This change really makes our code base more consistent and safer > > to change later. > > Did you consider expanding our tests along this change?
+1 Thanks for this improvement, Anna! I would also suggest to add specific test cases to check this behavior. - Boglarka ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54699/#review159003 ----------------------------------------------------------- On Dec. 13, 2016, 2:29 p.m., Anna Szonyi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54699/ > ----------------------------------------------------------- > > (Updated Dec. 13, 2016, 2:29 p.m.) > > > 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 > >
