> On Oct. 23, 2014, 2:52 p.m., Abraham Elmahrek wrote: > > common/src/main/java/org/apache/sqoop/json/ConnectorBean.java, line 111 > > <https://reviews.apache.org/r/27095/diff/3/?file=731022#file731022line111> > > > > Is there going to be documentation for all of the changes to the JSON > > structure? > > Veena Basavaraj wrote: > Yes, there is a ticket for it.
https://issues.apache.org/jira/browse/SQOOP-1518 > On Oct. 23, 2014, 2:52 p.m., Abraham Elmahrek wrote: > > common/src/main/java/org/apache/sqoop/json/ConnectorBean.java, lines 20-21 > > <https://reviews.apache.org/r/27095/diff/3/?file=731022#file731022line20> > > > > NIT: Use full names instead of .*? > > Veena Basavaraj wrote: > I spoke to Jarcec on the derby schema stuff, it is a pain with the merge > always. Is there a performance implication doing *? > > Abraham Elmahrek wrote: > It's more of a clutter thing I guess. Traditionally we've avoided using > wildcard imports. Unless every thing from the global space of each of those > packages is being imported, it's probably less cluttered to do use > non-wildcard imports. Even if the entirety of the package is being used, > there's always a chance that new items will be added that won't be used in > the future as well. hmm... not sure if it matters, I can change it back - Veena ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27095/#review58112 ----------------------------------------------------------- On Oct. 23, 2014, 1:10 p.m., Veena Basavaraj wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27095/ > ----------------------------------------------------------- > > (Updated Oct. 23, 2014, 1:10 p.m.) > > > 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 > > Note: Sqoop Client changes to support cname will be another RB, right now it > will support cid. > > > 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/model/MConfigList.java 8747b55 > common/src/main/java/org/apache/sqoop/model/MDriverConfig.java 679859a > common/src/main/java/org/apache/sqoop/model/MFromConfig.java 1b450d6 > common/src/main/java/org/apache/sqoop/model/MLinkConfig.java 318b63c > common/src/main/java/org/apache/sqoop/model/MToConfig.java b4fbe41 > 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 > common/src/test/java/org/apache/sqoop/model/TestMConfigList.java 9b60055 > 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 > >
