Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Sqoop, mail # dev - Review Request: SQOOP-846 Provide Netezza connector


+
Venkat Ranganathan 2013-02-21, 16:49
+
Venkat Ranganathan 2013-02-22, 07:58
+
Jarek Cecho 2013-02-24, 22:16
+
Venkat Ranganathan 2013-02-25, 19:00
+
Venkat Ranganathan 2013-02-27, 19:54
+
Venkat Ranganathan 2013-02-25, 19:43
+
Venkat Ranganathan 2013-02-25, 19:46
+
Venkat Ranganathan 2013-02-25, 19:50
+
Venkat Ranganathan 2013-02-25, 20:53
+
Venkat Ranganathan 2013-02-25, 22:02
+
Jarek Cecho 2013-02-27, 01:01
+
Venkat Ranganathan 2013-02-28, 05:04
+
Venkat Ranganathan 2013-02-28, 05:10
+
Venkat Ranganathan 2013-02-27, 23:48
+
Venkat Ranganathan 2013-02-28, 01:53
+
Venkat Ranganathan 2013-02-28, 08:06
+
Jarek Cecho 2013-03-02, 19:26
+
Jarek Cecho 2013-03-02, 23:02
+
Venkat Ranganathan 2013-03-02, 22:52
Copy link to this message
-
Re: Review Request: SQOOP-846 Provide Netezza connector
Venkat Ranganathan 2013-03-05, 01:26


> On March 2, 2013, 7:26 p.m., Jarek Cecho wrote:
> > Hi Venkat,
> > thank you very much for incorporating all my suggestions. I believe that we're almost there! I do have just couple of additional high level notes and few comments (mostly just nits):
> >
> > 1) Tests are passing for me - it was something specific to my environment. Thank you for checking though!
> >
> > 2) The docs are not compiling for me:
> >
> >      [exec] asciidoc: ERROR: connectors.txt: line 288: [blockdef-listing] missing closing delimiter
> >
> > 3) I would like to see more tests, especially about testing the various extra arguments. But I'm open to resolve that in follow up jira. No need to postpone this task even further.
>
> Venkat Ranganathan wrote:
>     Thanks Jarek for reviewing and feedback.
>    
>     1)  Great - I could run it with NPS 6 and 7.  And glad that it is now passing for you.  
>     2)  Fixed the docs.
>     3)  I agree.   I have couple of manual tests but will integrate them by filing followon JIRAs
>    
>     Will update new patch after testing and making sure the documentation is in order also
>    
>     Thanks
>     Venkat
>
> Jarek Cecho wrote:
>     Thank you for your quick feedback sir!

Hi Jarek

Sorry I missed marking couple of the changes as fixed.   But the patch has not changed (it was already part of the patch)

Thanks

Venkat
- Venkat
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9543/#review17303
-----------------------------------------------------------
On March 3, 2013, 5:16 a.m., Venkat Ranganathan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9543/
> -----------------------------------------------------------
>
> (Updated March 3, 2013, 5:16 a.m.)
>
>
> Review request for Sqoop and Jarek Cecho.
>
>
> Description
> -------
>
> This addresses SQOOP-846 (provide a Netezza connector)
>
>
> This addresses bug SQOOP-846.
>     https://issues.apache.org/jira/browse/SQOOP-846
>
>
> Diffs
> -----
>
>   src/docs/user/connectors.txt 7dd2a2e
>   src/java/org/apache/sqoop/lib/DelimiterSet.java 4e9bcab
>   src/java/org/apache/sqoop/manager/DefaultManagerFactory.java 54eb258
>   src/java/org/apache/sqoop/manager/DirectNetezzaManager.java PRE-CREATION
>   src/java/org/apache/sqoop/manager/MySQLUtils.java ef18818
>   src/java/org/apache/sqoop/manager/NetezzaManager.java PRE-CREATION
>   src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaDBDataSliceSplitter.java PRE-CREATION
>   src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableExportMapper.java PRE-CREATION
>   src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableImportMapper.java PRE-CREATION
>   src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableRecordExportMapper.java PRE-CREATION
>   src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableTextExportMapper.java PRE-CREATION
>   src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaJDBCStatementRunner.java PRE-CREATION
>   src/java/org/apache/sqoop/mapreduce/netezza/NetezzaDataDrivenDBInputFormat.java PRE-CREATION
>   src/java/org/apache/sqoop/mapreduce/netezza/NetezzaExternalTableExportJob.java PRE-CREATION
>   src/java/org/apache/sqoop/mapreduce/netezza/NetezzaExternalTableImportJob.java PRE-CREATION
>   src/java/org/apache/sqoop/mapreduce/netezza/NetezzaExternalTableInputFormat.java PRE-CREATION
>   src/java/org/apache/sqoop/mapreduce/netezza/NetezzaExternalTableInputSplit.java PRE-CREATION
>   src/test/com/cloudera/sqoop/manager/DirectNetezzaExportManualTest.java PRE-CREATION
>   src/test/com/cloudera/sqoop/manager/NetezzaImportManualTest.java PRE-CREATION
>   src/test/com/cloudera/sqoop/manager/NetezzaTestUtils.java PRE-CREATION
>
> Diff: https://reviews.apache.org/r/9543/diff/
>
>
> Testing
+
Venkat Ranganathan 2013-03-03, 05:16