----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50655/#review155806 -----------------------------------------------------------
Ship it! Thanks for fixing it! - Erzsebet Szilagyi On Oct. 6, 2016, 11:26 a.m., Benjamin BONNET wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50655/ > ----------------------------------------------------------- > > (Updated Oct. 6, 2016, 11:26 a.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/NetezzaExternalTableExportMapper.java > aa058d1 > > src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableImportMapper.java > 2efea53 > > src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaJDBCStatementRunner.java > cedfd23 > > 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 > >
