----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9076/#review15945 -----------------------------------------------------------
Ship it! Ship It! - Cheolsoo Park On Jan. 31, 2013, 5 p.m., Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9076/ > ----------------------------------------------------------- > > (Updated Jan. 31, 2013, 5 p.m.) > > > Review request for Sqoop. > > > Description > ------- > > Problem is that method File.list() in java might return null in addition to > empty array. I've add simple check that will throw more informative exception > in case that configuration path is not valid. > > > This addresses bug SQOOP-851. > https://issues.apache.org/jira/browse/SQOOP-851 > > > Diffs > ----- > > > submission/mapreduce/src/main/java/org/apache/sqoop/submission/mapreduce/MapreduceSubmissionEngine.java > bcd81249c15c83857352af4aaf247dfc2dce926a > > Diff: https://reviews.apache.org/r/9076/diff/ > > > Testing > ------- > > Unit tests seems to be passing and I've tested simple import on real cluster. > > > Thanks, > > Jarek Cecho > >
