sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jarek Cecho" <jar...@apache.org>
Subject Re: Review Request 13445: Fix for SQOOP-1167
Date Mon, 09 Sep 2013 15:19:05 GMT

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

Ship it!


Ship It!

- Jarek Cecho


On Sept. 7, 2013, 6:08 a.m., Venkat Ranganathan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13445/
> -----------------------------------------------------------
> 
> (Updated Sept. 7, 2013, 6:08 a.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-1167
>     https://issues.apache.org/jira/browse/SQOOP-1167
> 
> 
> Repository: sqoop-trunk
> 
> 
> Description
> -------
> 
> This patch implements the following 
> 
> Enhance HCat support to allow direct mode connectors by
>    Creating helpers for export and import hcat mappers and refactor hcat mappers to use
that
>    Adding additional method for connection managers to declare their ability to exploit
this feature
>    Make the detection of compatibility between hcat and direct mode managers after connection
managers are created
>    As an example usecase, fix the netezza implementation to
>        Abstract the Netezza direct mode mappers and add hcat support
>        Fix Netezza connector implementation issues etc
>    Add documentation
>    Add Netezza tests to third party test suite
>    Move Netezza tests to org.apache.namespace to be consistent with requirements for
newly added tests
> 
> 
> Diffs
> -----
> 
>   build.xml 1a21288 
>   src/docs/user/hcatalog.txt b8e495e 
>   src/java/org/apache/sqoop/manager/ConnManager.java f4b22f9 
>   src/java/org/apache/sqoop/manager/DirectNetezzaManager.java 4f36bf6 
>   src/java/org/apache/sqoop/mapreduce/ExportJobBase.java d0be570 
>   src/java/org/apache/sqoop/mapreduce/ImportJobBase.java ab7f21e 
>   src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableExportMapper.java
22b7af5 
>   src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableHCatExportMapper.java
PRE-CREATION 
>   src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableHCatImportMapper.java
PRE-CREATION 
>   src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableImportMapper.java
bcdc9e1 
>   src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableTextImportMapper.java
PRE-CREATION 
>   src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaJDBCStatementRunner.java 3a5df40

>   src/java/org/apache/sqoop/mapreduce/hcat/SqoopHCatExportHelper.java PRE-CREATION 
>   src/java/org/apache/sqoop/mapreduce/hcat/SqoopHCatExportMapper.java 539cedf 
>   src/java/org/apache/sqoop/mapreduce/hcat/SqoopHCatImportHelper.java PRE-CREATION 
>   src/java/org/apache/sqoop/mapreduce/hcat/SqoopHCatImportMapper.java 4f0ff1b 
>   src/java/org/apache/sqoop/mapreduce/netezza/NetezzaExternalTableExportJob.java 7caf9be

>   src/java/org/apache/sqoop/mapreduce/netezza/NetezzaExternalTableImportJob.java 0f7c1b0

>   src/java/org/apache/sqoop/tool/BaseSqoopTool.java 0eca991 
>   src/test/com/cloudera/sqoop/ThirdPartyTests.java 0a080b6 
>   src/test/com/cloudera/sqoop/manager/DirectNetezzaExportManualTest.java aace924 
>   src/test/com/cloudera/sqoop/manager/NetezzaExportManualTest.java 43dd254 
>   src/test/com/cloudera/sqoop/manager/NetezzaImportManualTest.java 86f5bdd 
>   src/test/com/cloudera/sqoop/manager/NetezzaTestUtils.java 4bf05b8 
>   src/test/org/apache/sqoop/hcat/HCatalogExportTest.java 77bafcc 
>   src/test/org/apache/sqoop/hcat/HCatalogImportTest.java 293015e 
>   src/test/org/apache/sqoop/hcat/HCatalogTestUtils.java ddae5f5 
>   src/test/org/apache/sqoop/hcat/TestHCatalogBasic.java da803d0 
>   src/test/org/apache/sqoop/manager/netezza/DirectNetezzaExportManualTest.java PRE-CREATION

>   src/test/org/apache/sqoop/manager/netezza/DirectNetezzaHCatExportManualTest.java PRE-CREATION

>   src/test/org/apache/sqoop/manager/netezza/DirectNetezzaHCatImportManualTest.java PRE-CREATION

>   src/test/org/apache/sqoop/manager/netezza/NetezzaExportManualTest.java PRE-CREATION

>   src/test/org/apache/sqoop/manager/netezza/NetezzaImportManualTest.java PRE-CREATION

>   src/test/org/apache/sqoop/manager/netezza/NetezzaTestUtils.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/13445/diff/
> 
> 
> Testing
> -------
> 
> Add additional tests for testing this feature.   Ran all tests and all of them passed
> 
> 
> Thanks,
> 
> Venkat Ranganathan
> 
>


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