----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28676/#review64658 -----------------------------------------------------------
Ship it! LGTM. Non-binding +1. - Gwen Shapira On Dec. 10, 2014, 9:13 p.m., Veena Basavaraj wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28676/ > ----------------------------------------------------------- > > (Updated Dec. 10, 2014, 9:13 p.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-1839 > https://issues.apache.org/jira/browse/SQOOP-1839 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > see jira > > > Diffs > ----- > > common/src/main/java/org/apache/sqoop/json/SubmissionBean.java e926f02 > common/src/main/java/org/apache/sqoop/model/MSubmission.java 2648712 > common/src/main/java/org/apache/sqoop/model/SubmissionError.java > PRE-CREATION > common/src/test/java/org/apache/sqoop/json/TestSubmissionBean.java d07eda9 > core/src/main/java/org/apache/sqoop/driver/JobManager.java f4f5561 > core/src/main/java/org/apache/sqoop/driver/JobRequest.java eed79a5 > core/src/main/java/org/apache/sqoop/driver/SubmissionEngine.java 3a32e9f > docs/src/site/sphinx/RESTAPI.rst c0c236e > > repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java > c278406 > > repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestSubmissionHandling.java > e2e8073 > shell/src/main/java/org/apache/sqoop/shell/ShowSubmissionFunction.java > 4a0f476 > shell/src/main/java/org/apache/sqoop/shell/utils/SubmissionDisplayer.java > 0e2a38d > > submission/mapreduce/src/main/java/org/apache/sqoop/submission/mapreduce/MapreduceSubmissionEngine.java > 631ceca > test/src/main/java/org/apache/sqoop/test/testcases/ConnectorTestCase.java > 4f5f52a > > Diff: https://reviews.apache.org/r/28676/diff/ > > > Testing > ------- > > > Thanks, > > Veena Basavaraj > >
