----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50655/#review146346 -----------------------------------------------------------
src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableImportMapper.java <https://reviews.apache.org/r/50655/#comment212730> I'm curious about why you've removed this LOG line? - Erzsebet Szilagyi On Aug. 1, 2016, 4:13 p.m., Benjamin BONNET wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50655/ > ----------------------------------------------------------- > > (Updated Aug. 1, 2016, 4:13 p.m.) > > > Review request for Sqoop, David Robson, Jarek Cecho, and Kathleen Ting. > > > Repository: sqoop-trunk > > > Description > ------- > > See on JIRA https://issues.apache.org/jira/browse/SQOOP-2991 > > > Diffs > ----- > > > src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableImportMapper.java > 2efea53afd7eb13ee7a2af48fc4278ac65ff251f > > src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaJDBCStatementRunner.java > cedfd235c7c0fc764b24123cf87746d85f87b888 > > Diff: https://reviews.apache.org/r/50655/diff/ > > > Testing > ------- > > Tested with requests that throw errors on Netezza side : importing with a > user that has not enough rights to create an external table. Without patch, > import fails but map reduce job never ends, with patch, import fails, map > reduce job ends with an IOException. > > > Thanks, > > Benjamin BONNET > >
