> 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 .*?
I spoke to Jarcec on the derby schema stuff, it is a pain with the merge always. Is there a performance implication doing *? > 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? Yes, there is a ticket for it. > On Oct. 23, 2014, 2:52 p.m., Abraham Elmahrek wrote: > > common/src/main/java/org/apache/sqoop/json/LinkBean.java, lines 115-122 > > <https://reviews.apache.org/r/27095/diff/3/?file=731026#file731026line115> > > > > If we've split up the APIs for fetch bundles and resources, then this > > line of code is unnecessary. Otherwise, the bundles should probably be > > shipped with responses to resource requests. yes the link config GET resource will add this( it is nother ticket) it is in the design pdf attached v1/connector/{cname}/link GET A connector has zero or more links.Hence the api list the parent/child relationship in the REST api Gets the config params associated to create a link object for this connector ( used in the context of UI in Hue to show the list of params to fill out). It could also be an api on the config resource ( still up for discussion ) the bundles are shipped with connectors/ drivers that are configurables as well. > On Oct. 23, 2014, 2:52 p.m., Abraham Elmahrek wrote: > > common/src/main/java/org/apache/sqoop/model/MConfigList.java, lines 87-88 > > <https://reviews.apache.org/r/27095/diff/3/?file=731033#file731033line87> > > > > NIT: Braces? fixed > On Oct. 23, 2014, 2:52 p.m., Abraham Elmahrek wrote: > > server/src/main/webapp/WEB-INF/web.xml, lines 67-71 > > <https://reviews.apache.org/r/27095/diff/3/?file=731056#file731056line67> > > > > Does this need to be redefined? it connector and connectors yes 2 separate ones. - 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 > >
