----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27095/ -----------------------------------------------------------
Review request for Sqoop. Repository: sqoop-sqoop2 Description ------- see JIRA https://issues.apache.org/jira/browse/SQOOP-1514 https://issues.apache.org/jira/browse/SQOOP-1512 and the parent ticket: https://issues.apache.org/jira/browse/SQOOP-1509 It modifies the driver api and connector api. It adds a configurable api with java docs on how to use it. The relevant constants have been moved to their respective classes There are WS, that will be cleaned up soon Diffs ----- client/src/main/java/org/apache/sqoop/client/SqoopClient.java bffb599 client/src/main/java/org/apache/sqoop/client/request/DriverResourceRequest.java PRE-CREATION common/src/main/java/org/apache/sqoop/json/ConfigurableBean.java PRE-CREATION common/src/main/java/org/apache/sqoop/json/ConnectorBean.java d12b6b0 common/src/main/java/org/apache/sqoop/json/DriverBean.java 90cdbef common/src/main/java/org/apache/sqoop/json/JobBean.java 8f42edb common/src/main/java/org/apache/sqoop/json/JsonBean.java 8189259 common/src/main/java/org/apache/sqoop/json/LinkBean.java af0fc9d common/src/main/java/org/apache/sqoop/json/util/ConfigInputConstants.java PRE-CREATION common/src/main/java/org/apache/sqoop/json/util/ConfigInputSerialization.java PRE-CREATION common/src/main/java/org/apache/sqoop/json/util/ConfigParamBundleSerialization.java PRE-CREATION common/src/main/java/org/apache/sqoop/json/util/ConfigSerialization.java cec46f6 common/src/main/java/org/apache/sqoop/json/util/ResourceBundleSerialization.java f27d81d common/src/main/java/org/apache/sqoop/json/util/SchemaSerialization.java 1e6da6d common/src/main/java/org/apache/sqoop/utils/MapResourceBundle.java b0a15e7 common/src/test/java/org/apache/sqoop/json/TestLinkBean.java ac07137 common/src/test/java/org/apache/sqoop/json/util/TestConfigSerialization.java 4f0c84d core/src/main/java/org/apache/sqoop/connector/ConnectorManager.java 0369b4d core/src/main/java/org/apache/sqoop/driver/Driver.java 6942891 core/src/main/java/org/apache/sqoop/driver/DriverConfigValidator.java 0d9a9b8 core/src/main/java/org/apache/sqoop/driver/DriverError.java 56ef9bb core/src/main/java/org/apache/sqoop/repository/JdbcRepository.java d7b526a server/src/main/java/org/apache/sqoop/handler/ConnectorRequestHandler.java c50e029 server/src/main/java/org/apache/sqoop/handler/DriverConfigRequestHandler.java aa773a9 server/src/main/java/org/apache/sqoop/handler/DriverRequestHandler.java PRE-CREATION server/src/main/java/org/apache/sqoop/handler/JobRequestHandler.java 0cd5acb server/src/main/java/org/apache/sqoop/server/v1/ConfigurableServlet.java PRE-CREATION server/src/main/java/org/apache/sqoop/server/v1/ConnectorServlet.java d24ac10 server/src/main/java/org/apache/sqoop/server/v1/ConnectorsServlet.java PRE-CREATION server/src/main/java/org/apache/sqoop/server/v1/DriverConfigServlet.java c2b1f9f server/src/main/java/org/apache/sqoop/server/v1/DriverServlet.java PRE-CREATION server/src/main/webapp/WEB-INF/web.xml d31120a tools/src/main/java/org/apache/sqoop/tools/tool/RepositoryDumpTool.java 819cf6a Diff: https://reviews.apache.org/r/27095/diff/ Testing ------- yes Thanks, Veena Basavaraj
