----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42129/ -----------------------------------------------------------
Review request for Sqoop. Repository: sqoop-sqoop2 Description ------- Currently, SqoopInfrastructureProvider is started only once for test suite, and the default SqoopMiniCluster is JettySqoopMiniCluster. With the different test connectors, SqoopInfrastructureProvider should be started with different SqoopMiniCluster. A restart feature should be added to support this. Diffs ----- test/src/main/java/org/apache/sqoop/test/infrastructure/InfraProvider.java PRE-CREATION test/src/main/java/org/apache/sqoop/test/infrastructure/InfraProviderWrapper.java PRE-CREATION test/src/main/java/org/apache/sqoop/test/infrastructure/Infrastructure.java f3db7ad test/src/main/java/org/apache/sqoop/test/infrastructure/SqoopTestCase.java becfa6b test/src/main/java/org/apache/sqoop/test/infrastructure/providers/SqoopInfrastructureProvider.java 4d51ed6 test/src/main/java/org/apache/sqoop/test/testcases/ConnectorClasspathTestCase.java 6db1db8 test/src/main/java/org/apache/sqoop/test/testcases/ConnectorLoadingTestCase.java PRE-CREATION test/src/main/java/org/apache/sqoop/test/testcases/ConnectorTestCase.java f071786 test/src/test/java/org/apache/sqoop/integration/connector/hdfs/AppendModeTest.java 6885525 test/src/test/java/org/apache/sqoop/integration/connector/hdfs/FromHDFSToHDFSTest.java c6ce1e8 test/src/test/java/org/apache/sqoop/integration/connector/hdfs/HdfsIncrementalReadTest.java 37306e2 test/src/test/java/org/apache/sqoop/integration/connector/hdfs/InformalJobNameExecuteTest.java 0e14b7a test/src/test/java/org/apache/sqoop/integration/connector/hdfs/OutputDirectoryTest.java e5e3d26 test/src/test/java/org/apache/sqoop/integration/connector/hdfs/S3Test.java c857699 test/src/test/java/org/apache/sqoop/integration/connector/hive/FromRDBMSToKiteHiveTest.java ec9f733 test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/AllTypesTest.java 9c0ee84 test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/FromHDFSToRDBMSTest.java 933bc08 test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/FromRDBMSToHDFSTest.java 7e66091 test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/IncrementalReadTest.java 83012eb test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/PartitionerTest.java e5e886e test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableStagedRDBMSTest.java 890fc10 test/src/test/java/org/apache/sqoop/integration/connector/kafka/FromHDFSToKafkaTest.java 5e349d1 test/src/test/java/org/apache/sqoop/integration/connector/kafka/FromRDBMSToKafkaTest.java 9ae1334 test/src/test/java/org/apache/sqoop/integration/connector/kite/FromRDBMSToKiteTest.java 10f3614 test/src/test/java/org/apache/sqoop/integration/connectorloading/BlacklistedConnectorTest.java 66c20a1 test/src/test/java/org/apache/sqoop/integration/connectorloading/ConnectorClasspathIsolationTest.java 5b95631 test/src/test/java/org/apache/sqoop/integration/repository/derby/upgrade/DerbyRepositoryUpgradeTest.java 44f4a5b test/src/test/java/org/apache/sqoop/integration/server/InformalObjectNameTest.java 5e204c8 test/src/test/java/org/apache/sqoop/integration/server/ShowJobInOrderTest.java 49e26a2 test/src/test/java/org/apache/sqoop/integration/server/SubmissionWithDisabledModelObjectsTest.java 9adebea test/src/test/java/org/apache/sqoop/integration/server/VersionTest.java 76002a5 test/src/test/java/org/apache/sqoop/integration/server/rest/RestTest.java 71384fb test/src/test/java/org/apache/sqoop/integration/tools/RepositoryDumpLoadToolTest.java cce2c6c Diff: https://reviews.apache.org/r/42129/diff/ Testing ------- Thanks, Colin Ma
