-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39396/
-----------------------------------------------------------
(Updated Oct. 19, 2015, 3:03 a.m.)
Review request for Sqoop.
Bugs: SQOOP-2621
https://issues.apache.org/jira/browse/SQOOP-2621
Repository: sqoop-sqoop2
Description
-------
Provide classpath isolation for connectors and also integration test.
Diffs (updated)
-----
common/src/main/java/org/apache/sqoop/error/code/ConnectorError.java 2f17d95
common/src/main/java/org/apache/sqoop/utils/ClassUtils.java ed68988
common/src/main/java/org/apache/sqoop/utils/ContextUtils.java 25d6fcb
connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcConnector.java
113465a
connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsConnector.java
7e7c022
connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteConnector.java
ca860b1
connector/connector-sdk/src/main/java/org/apache/sqoop/connector/spi/SqoopConnector.java
85ba8be
connector/connector-sdk/src/main/java/org/apache/sqoop/job/etl/From.java
e9d2abe
connector/connector-sdk/src/main/java/org/apache/sqoop/job/etl/Partition.java
c4664a6
core/src/main/java/org/apache/sqoop/connector/ConnectorHandler.java 1899bb7
core/src/main/java/org/apache/sqoop/connector/ConnectorManager.java 285e893
core/src/main/java/org/apache/sqoop/connector/ConnectorManagerUtils.java
522ed08
core/src/main/java/org/apache/sqoop/connector/ConnectorUtils.java
PRE-CREATION
core/src/main/java/org/apache/sqoop/core/ConfigurationConstants.java f49fca3
core/src/main/java/org/apache/sqoop/core/SqoopConfiguration.java 3c2f67e
core/src/main/java/org/apache/sqoop/driver/JobManager.java 0d230f9
core/src/main/java/org/apache/sqoop/driver/JobRequest.java cfa45b2
core/src/test/java/org/apache/sqoop/connector/TestConnectorManagerUtils.java
fd2d1b4
core/src/test/java/org/apache/sqoop/connector/TestConnectorUtils.java
PRE-CREATION
dist/src/main/conf/sqoop.properties 7c1ec18
execution/mapreduce/src/main/java/org/apache/sqoop/execution/mapreduce/MapreduceExecutionEngine.java
c8d210e
execution/mapreduce/src/main/java/org/apache/sqoop/job/MRJobConstants.java
b7aa8c6
execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/MRConfigurationUtils.java
1e1b237
execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/MRUtils.java
PRE-CREATION
execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopDestroyerExecutor.java
b3c1ce8
execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopInputFormat.java
67189a1
execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopMapper.java
937ef5a
execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopNullOutputFormat.java
88ab98e
execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopOutputFormatLoadExecutor.java
d94b658
execution/mapreduce/src/test/java/org/apache/sqoop/job/TestMapReduce.java
2463643
execution/mapreduce/src/test/java/org/apache/sqoop/job/TestMatching.java
d0b41d1
execution/mapreduce/src/test/java/org/apache/sqoop/job/mr/TestMRConfigurationUtils.java
6a12d47
repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
0302150
submission/mapreduce/src/main/java/org/apache/sqoop/submission/mapreduce/MapreduceSubmissionEngine.java
f396783
test/src/main/java/org/apache/sqoop/test/testcases/TestClasspathBase.java
PRE-CREATION
test/src/test/java/org/apache/sqoop/integration/classpath/ClasspathTest.java
393b800
test/src/test/java/org/apache/sqoop/integration/classpath/ConnectorClasspathIsolationTest.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/from/TestClasspathIsolation.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/from/TestExtractor.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/from/TestFromConnector.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/from/TestFromDestroyer.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/from/TestFromInitializer.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/from/TestFromJobConfiguration.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/from/TestFromLinkConfiguration.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/from/TestPartition.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/from/TestPartitioner.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/from/sqoopconnector.properties
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/to/TestClasspathIsolation.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/to/TestLoader.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/to/TestToConnector.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/to/TestToDestroyer.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/to/TestToInitializer.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/to/TestToJobConfiguration.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/to/TestToLinkConfiguration.java
PRE-CREATION
test/src/test/resources/TestConnectorClasspathIsolation/to/sqoopconnector.properties
PRE-CREATION
Diff: https://reviews.apache.org/r/39396/diff/
Testing
-------
Thanks,
Dian Fu