----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26410/#review62218 -----------------------------------------------------------
Could you add a tests for the added functionality Nick? - Jarek Cecho On Oct. 7, 2014, 12:05 p.m., Nick White wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26410/ > ----------------------------------------------------------- > > (Updated Oct. 7, 2014, 12:05 p.m.) > > > Review request for Sqoop. > > > Repository: sqoop-trunk > > > Description > ------- > > Add --continue option to import-all-tables to ignore (but log) errors on any > single table. > > > Diffs > ----- > > src/java/org/apache/sqoop/SqoopOptions.java d16ccb3 > src/java/org/apache/sqoop/tool/BaseSqoopTool.java 498ad79 > src/java/org/apache/sqoop/tool/ImportAllTablesTool.java ab95959 > > Diff: https://reviews.apache.org/r/26410/diff/ > > > Testing > ------- > > > File Attachments > ---------------- > > sqoop-all-tables-continue.patch > > https://reviews.apache.org/media/uploaded/files/2014/10/07/3089a1e6-6129-476d-9203-e0ac0267fbef__sqoop-all-tables-continue.patch > > > Thanks, > > Nick White > >
