----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50655/#review159696 -----------------------------------------------------------
src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaJDBCStatementRunner.java (line 101) <https://reviews.apache.org/r/50655/#comment230718> Hey Benjamin, Could you please describe why are we using LF,CR characters here? I'm a bit concerned, because AFAIK this line ending form is only supported by Microsoft Windows systems. On the other hand (if I'm not mistaken) there it's in the opposite direction (so CR,LF). Are we sure that this will also work on Unix systems? (Like Linux, BSD, OSX, etc.) Or is Netezza using this quite platform dependent way to end lines? Thanks for the clarification! Attila - Attila Szabo On Oct. 6, 2016, 9: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, 9: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 > >
