sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Abraham Elmahrek" <...@cloudera.com>
Subject Re: Review Request 28470: SQOOP-1633: Unit/ Integration test
Date Tue, 02 Dec 2014 04:43:52 GMT

-----------------------------------------------------------
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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message