-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25491/#review52860
-----------------------------------------------------------



connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestLoader.java
<https://reviews.apache.org/r/25491/#comment92021>

    The codec sub set has been here intentionally as some of the codecs have 
dependencies on native libraries that we can't ensure on all build boxes which 
is causing unnecessary false alarms.
    
    I would suggest to either use the subset we had there before or add 
Assume.assumeTrue assertions to skip certain codecs if required native 
libraries are not available.
    
    I see following exceptions on machine that don't have any hadoop native 
libraries available:
    
      testLoader[7](org.apache.sqoop.connector.hdfs.TestLoader): SequenceFile 
doesn't work with GzipCodec without native-hadoop code!
      testLoader[10](org.apache.sqoop.connector.hdfs.TestLoader): 
GENERIC_HDFS_CONNECTOR_0003:Unable to load the specified class - 
com.hadoop.compression.lzo.LzoCodec
      testLoader[11](org.apache.sqoop.connector.hdfs.TestLoader): 
GENERIC_HDFS_CONNECTOR_0003:Unable to load the specified class - 
com.hadoop.compression.lzo.LzoCodec
      testLoader[12](org.apache.sqoop.connector.hdfs.TestLoader): native lz4 
library not available
      testLoader[13](org.apache.sqoop.connector.hdfs.TestLoader): native lz4 
library not available
      testLoader[14](org.apache.sqoop.connector.hdfs.TestLoader): 
org.apache.hadoop.util.NativeCodeLoader.buildSupportsSnappy()Z
      testLoader[15](org.apache.sqoop.connector.hdfs.TestLoader): 
org.apache.hadoop.util.NativeCodeLoader.buildSupportsSnappy()Z
      testLoader[16](org.apache.sqoop.connector.hdfs.TestLoader)
      testLoader[17](org.apache.sqoop.connector.hdfs.TestLoader)


- Jarek Cecho


On Sept. 10, 2014, 3:49 a.m., Abraham Elmahrek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25491/
> -----------------------------------------------------------
> 
> (Updated Sept. 10, 2014, 3:49 a.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-1506
>     https://issues.apache.org/jira/browse/SQOOP-1506
> 
> 
> Repository: sqoop-SQOOP-1367
> 
> 
> Description
> -------
> 
> commit 1d865164dca2dd7f359dff837013acbc3dfe9ace
> Author: Abraham Elmahrek <[email protected]>
> Date:   Mon Sep 8 22:33:42 2014 -0700
> 
>     SQOOP-1506: Sqoop2: From/To: Re-enable all tests
>     
>     Left out a couple in TestJdbcRepository.
>     Left out proper testing in TableStagedRDBMSTest.
>     Depends on SQOOP-1488.
>     Removed MapreduceExecutionEngineTest since it seems
>     to exist in HdfsConnectors.
> 
> :100644 100644 b5e7e61... 731f6cc... M  
> client/src/test/java/org/apache/sqoop/client/TestSqoopClient.java
> :100644 100644 a50c815... c0d678a... M  
> common/src/main/java/org/apache/sqoop/model/MValidatedElement.java
> :000000 100644 0000000... 9d11a2e... A  
> common/src/main/java/org/apache/sqoop/validation/validators/NullOrContains.java
> :000000 100644 0000000... 3c64346... A  
> common/src/test/java/org/apache/sqoop/validation/validators/TestNullOrContains.java
> :100644 100644 a25f344... db79acc... M  
> connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcPartitioner.java
> :100644 100644 ac72546... 72902a2... M  
> connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/configuration/FromTableForm.java
> :100644 100644 4831cf8... eb6fcf1... M  
> connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestToInitializer.java
> :100644 100644 5a924f9... 61676fe... M  
> connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsLoader.java
> :100644 000000 d4aaa0a... 0000000... D  
> connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/configuration/StorageType.java
> :100644 100644 79cf1f1... 644bbc2... M  
> connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestLoader.java
> :100644 100644 830606a... 897d3c7... M  
> core/src/main/java/org/apache/sqoop/framework/configuration/ConnectionConfiguration.java
> :100644 100644 90395ac... 69c1b56... M  
> core/src/test/java/org/apache/sqoop/framework/TestFrameworkValidator.java
> :100644 100644 f19e01c... 50daa62... M  
> core/src/test/java/org/apache/sqoop/repository/TestJdbcRepository.java
> :100644 100644 1d60ba3... 3065680... M  
> execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopMapper.java
> :100644 000000 f70e9bd... 0000000... D  
> execution/mapreduce/src/test/java/org/apache/sqoop/execution/mapreduce/MapreduceExecutionEngineTest.java
> :100644 100644 fae9fe8... 9d9b508... M  pom.xml
> :100644 100644 e2f1675... 9ecc9da... M  
> test/src/main/java/org/apache/sqoop/test/minicluster/TomcatSqoopMiniCluster.java
> :100644 100644 af0f299... 8a699f5... M  
> test/src/main/java/org/apache/sqoop/test/testcases/ConnectorTestCase.java
> :100644 100644 59c5f15... ec650e1... M  
> test/src/main/java/org/apache/sqoop/test/utils/HdfsUtils.java
> :000000 100644 0000000... c01aa80... A  
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/FromHDFSToRDBMSTest.java
> :000000 100644 0000000... f976f29... A  
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/FromRDBMSToHDFSTest.java
> :000000 100644 0000000... fac7e8b... A  
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/PartitionerTest.java
> :100644 000000 39b48d8... 0000000... D  
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableExportTest.java
> :100644 000000 9171b8e... 0000000... D  
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableImportTest.java
> :000000 100644 0000000... cb782c7... A  
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableStagedRDBMSTest.java
> :100644 000000 cb028bb... 0000000... D  
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/exports/TableStagedExportTest.java
> :100644 000000 a0a4022... 0000000... D  
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/imports/PartitionerTest.java
> :100644 100644 1a7a3a8... 3c01cb0... M  
> test/src/test/java/org/apache/sqoop/integration/server/SubmissionWithDisabledModelObjectsTest.java
> :100644 100644 5ebe95f... cea24b9... M  
> test/src/test/java/org/apache/sqoop/integration/server/VersionTest.java
> 
> 
> Diffs
> -----
> 
>   client/src/test/java/org/apache/sqoop/client/TestSqoopClient.java b5e7e61 
>   common/src/main/java/org/apache/sqoop/model/MValidatedElement.java a50c815 
>   
> common/src/main/java/org/apache/sqoop/validation/validators/NullOrContains.java
>  PRE-CREATION 
>   
> common/src/test/java/org/apache/sqoop/validation/validators/TestNullOrContains.java
>  PRE-CREATION 
>   
> connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcPartitioner.java
>  a25f344 
>   
> connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/configuration/FromTableForm.java
>  ac72546 
>   
> connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestToInitializer.java
>  4831cf8 
>   
> connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsLoader.java
>  5a924f9 
>   
> connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/configuration/StorageType.java
>  d4aaa0a 
>   
> connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestLoader.java
>  79cf1f1 
>   
> core/src/main/java/org/apache/sqoop/framework/configuration/ConnectionConfiguration.java
>  830606a 
>   core/src/test/java/org/apache/sqoop/framework/TestFrameworkValidator.java 
> 90395ac 
>   core/src/test/java/org/apache/sqoop/repository/TestJdbcRepository.java 
> f19e01c 
>   execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopMapper.java 
> 1d60ba3 
>   
> execution/mapreduce/src/test/java/org/apache/sqoop/execution/mapreduce/MapreduceExecutionEngineTest.java
>  f70e9bd 
>   pom.xml fae9fe8 
>   
> test/src/main/java/org/apache/sqoop/test/minicluster/TomcatSqoopMiniCluster.java
>  e2f1675 
>   test/src/main/java/org/apache/sqoop/test/testcases/ConnectorTestCase.java 
> af0f299 
>   test/src/main/java/org/apache/sqoop/test/utils/HdfsUtils.java 59c5f15 
>   
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/FromHDFSToRDBMSTest.java
>  PRE-CREATION 
>   
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/FromRDBMSToHDFSTest.java
>  PRE-CREATION 
>   
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/PartitionerTest.java
>  PRE-CREATION 
>   
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableExportTest.java
>  39b48d8 
>   
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableImportTest.java
>  9171b8e 
>   
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableStagedRDBMSTest.java
>  PRE-CREATION 
>   
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/exports/TableStagedExportTest.java
>  cb028bb 
>   
> test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/imports/PartitionerTest.java
>  a0a4022 
>   
> test/src/test/java/org/apache/sqoop/integration/server/SubmissionWithDisabledModelObjectsTest.java
>  1a7a3a8 
>   test/src/test/java/org/apache/sqoop/integration/server/VersionTest.java 
> 5ebe95f 
> 
> Diff: https://reviews.apache.org/r/25491/diff/
> 
> 
> Testing
> -------
> 
> mvn integration-test passes.
> 
> 
> Thanks,
> 
> Abraham Elmahrek
> 
>

Reply via email to