----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41699/#review111831 -----------------------------------------------------------
Ship it! Ship It! - Jarek Cecho On Dec. 24, 2015, 3:35 a.m., Colin Ma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41699/ > ----------------------------------------------------------- > > (Updated Dec. 24, 2015, 3:35 a.m.) > > > Review request for Sqoop. > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Clear useless method after remove id from public interface > > > Diffs > ----- > > client/src/main/java/org/apache/sqoop/client/SqoopClient.java bac7fe8 > core/src/main/java/org/apache/sqoop/connector/ConnectorManager.java 5663445 > core/src/main/java/org/apache/sqoop/driver/JobManager.java 39a0260 > core/src/main/java/org/apache/sqoop/repository/JdbcRepository.java 952ce5d > core/src/main/java/org/apache/sqoop/repository/Repository.java 975e2aa > core/src/test/java/org/apache/sqoop/driver/TestJobManager.java ee88ddc > > repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/DerbyTestCase.java > e7b4322 > > repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestConnectorHandling.java > fce6e83 > server/src/main/java/org/apache/sqoop/handler/ConnectorRequestHandler.java > efe9bef > server/src/main/java/org/apache/sqoop/handler/HandlerUtils.java 1eee2d2 > server/src/main/java/org/apache/sqoop/server/v1/ConfigurableServlet.java > 7bd1901 > server/src/main/java/org/apache/sqoop/server/v1/ConnectorServlet.java > 5daebc1 > server/src/main/java/org/apache/sqoop/server/v1/JobServlet.java 7767132 > server/src/main/java/org/apache/sqoop/server/v1/LinkServlet.java d803435 > shell/src/main/java/org/apache/sqoop/shell/ShowSubmissionFunction.java > aeac76a > shell/src/test/java/org/apache/sqoop/shell/TestCloneCommand.java ec6f18f > shell/src/test/java/org/apache/sqoop/shell/TestShowCommand.java 8f8d3db > shell/src/test/java/org/apache/sqoop/shell/TestUpdateCommand.java 4b1f5ad > test/src/main/java/org/apache/sqoop/test/infrastructure/SqoopTestCase.java > dbe2195 > > Diff: https://reviews.apache.org/r/41699/diff/ > > > Testing > ------- > > > Thanks, > > Colin Ma > >
