> On Nov. 8, 2016, 5:08 p.m., Attila Szabo wrote: > > Ship It!
Thank you Bogi for applying the changes! It really made it better, nice job! - Attila ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53134/#review155304 ----------------------------------------------------------- On Nov. 8, 2016, 2:53 p.m., Boglarka Egyed wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53134/ > ----------------------------------------------------------- > > (Updated Nov. 8, 2016, 2:53 p.m.) > > > Review request for Sqoop and Attila Szabo. > > > Bugs: SQOOP-3034 > https://issues.apache.org/jira/browse/SQOOP-3034 > > > Repository: sqoop-trunk > > > Description > ------- > > HBase import should fail fast if using anything other than as-textfile as > those are ignored in the current implementation, however, the user is not > informed about it and the HBase import performs silently as a textfile. > > > Diffs > ----- > > src/java/org/apache/sqoop/tool/BaseSqoopTool.java > 13a9697b2d94e3846f86e1b605cfff3d01dd9347 > src/test/org/apache/sqoop/tool/ImportToolValidateOptionsTest.java > 503a86340f5ae33c8eb0a27b8450b2af0a342db9 > > Diff: https://reviews.apache.org/r/53134/diff/ > > > Testing > ------- > > Test cases have been added for option validation check. > > > Thanks, > > Boglarka Egyed > >
