----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56914/#review166531 -----------------------------------------------------------
Ship it! - Venkat Ranganathan On Feb. 21, 2017, 9:12 p.m., Suraj Nayak wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56914/ > ----------------------------------------------------------- > > (Updated Feb. 21, 2017, 9:12 p.m.) > > > Review request for Sqoop. > > > Repository: sqoop-trunk > > > Description > ------- > > Netezza Direct import Mapper is not sensitive to columns option. The > parameters passed to --columns C1,C2,C3,C4 are ignored by > NetezzaExternalTableImportMapper. > NetezzaExternalTableImportMapper on line 83 is based of output field names > instead of input field names. > Even when -D mapreduce.jdbc.output.field.names=C1,C2,C3,C4 is set, another > bug in NetezzaExternalTableImportMapper @ line 128 duplicates the first > column. This is due to the for loop initialization bug where variable i > should be initialized to 1 instead of 0. > > > Diffs > ----- > > > src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableImportMapper.java > 2efea53 > > Diff: https://reviews.apache.org/r/56914/diff/ > > > Testing > ------- > > Manually ran ant test. All test cases passed. Build successful. > > > Thanks, > > Suraj Nayak > >
