----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36175/#review92712 -----------------------------------------------------------
A couple of more comments. common/src/main/java/org/apache/sqoop/json/SubmissionBean.java (line 47) <https://reviews.apache.org/r/36175/#comment146947> Kill white space common/src/main/java/org/apache/sqoop/json/SubmissionBean.java (line 107) <https://reviews.apache.org/r/36175/#comment146948> Need to set JOB_ID and JOB. If JOB and JOB_ID is set when restoring, then JOB_ID would take precedence. - Abraham Elmahrek On July 23, 2015, 1:52 a.m., Dian Fu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36175/ > ----------------------------------------------------------- > > (Updated July 23, 2015, 1:52 a.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-1256 > https://issues.apache.org/jira/browse/SQOOP-1256 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Load repository dump into a different repository > > > Diffs > ----- > > common/src/main/java/org/apache/sqoop/json/SubmissionBean.java ad05057 > tools/src/main/java/org/apache/sqoop/tools/tool/JSONConstants.java 9301707 > tools/src/main/java/org/apache/sqoop/tools/tool/RepositoryDumpTool.java > 1027baf > tools/src/main/java/org/apache/sqoop/tools/tool/RepositoryLoadTool.java > a25ebe8 > > Diff: https://reviews.apache.org/r/36175/diff/ > > > Testing > ------- > > > Thanks, > > Dian Fu > >
