----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28470/#review63101 -----------------------------------------------------------
Seems good. Are we 100% on running the test suite in 2 different modes as we have now? If so, might be cool to log a follow up Jira to exclude certain tests from needing the KDC some how. pom.xml <https://reviews.apache.org/r/28470/#comment105269> <scope>test</scope> ? test/src/main/java/org/apache/sqoop/test/kdc/KdcRunner.java <https://reviews.apache.org/r/28470/#comment105759> Is this used any where? Or is it for future support? test/src/main/java/org/apache/sqoop/test/kdc/KdcRunner.java <https://reviews.apache.org/r/28470/#comment105273> rm -rf? test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/FromRDBMSToHDFSTest.java <https://reviews.apache.org/r/28470/#comment105272> Unused imports? test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/FromRDBMSToHDFSTest.java <https://reviews.apache.org/r/28470/#comment105270> spacing increased? test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/FromRDBMSToHDFSTest.java <https://reviews.apache.org/r/28470/#comment105271> same - Abraham Elmahrek On Nov. 26, 2014, 8:53 a.m., richard zhou wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28470/ > ----------------------------------------------------------- > > (Updated Nov. 26, 2014, 8:53 a.m.) > > > Review request for Sqoop. > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Unit/ Integration test > It is a initial patch, just for review. The Simple auth works. The Kerberos > auth does not work. Next patch will fix this bug. > > Use Simple auth: mvn integration-test > -Dsqoop.hadoop.runner.class=org.apache.sqoop.test.hadoop.HadoopLocalRunner > Use Kerberos auth: mvn integration-test > -Dsqoop.hadoop.runner.class=org.apache.sqoop.test.hadoop.HadoopLocalRunner > -Dorg.apache.sqoop.authentication.type=kerberos > > > Diffs > ----- > > pom.xml ddcc5379ce879781a0618275dbe7e9a56d331536 > test/pom.xml cafa250d583eaede3058c375c3341479eee50ad8 > test/src/main/java/org/apache/sqoop/test/kdc/KdcRunner.java PRE-CREATION > test/src/main/java/org/apache/sqoop/test/minicluster/SqoopMiniCluster.java > 4322b1cdb7841d2157d3fd1e4abf1b131dc94130 > test/src/main/java/org/apache/sqoop/test/testcases/TomcatTestCase.java > 5e1d564aef2c8f566f7baa781009328bd0ca5e20 > test/src/main/java/org/apache/sqoop/test/utils/ConfigurationUtils.java > PRE-CREATION > > test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/FromRDBMSToHDFSTest.java > e482ac598d7dc9265922c9d2b6ad29abeb0c7b50 > > Diff: https://reviews.apache.org/r/28470/diff/ > > > Testing > ------- > > > Thanks, > > richard zhou > >
