-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28139/
-----------------------------------------------------------
(Updated Nov. 17, 2014, 4 p.m.)
Review request for Sqoop.
Bugs: SQOOP-1749
https://issues.apache.org/jira/browse/SQOOP-1749
Repository: sqoop-sqoop2
Description (updated)
-------
see JIRA
Supporting Nested arrays is going to be dody logic
some of the assumptions here in parsing need more docs. Esp use of special
characters to parse nested arrays.
Welcome feedback if I can improve this more.
I am sure mores tests can be added.
Diffs
-----
common/src/main/java/org/apache/sqoop/schema/type/AbstractComplexListType.java
5d8f28b
common/src/main/java/org/apache/sqoop/schema/type/Array.java af13eb7
connector/connector-sdk/src/main/java/org/apache/sqoop/connector/idf/CSVIntermediateDataFormat.java
39a01c1
connector/connector-sdk/src/main/java/org/apache/sqoop/connector/idf/IntermediateDataFormat.java
5ef6fc6
connector/connector-sdk/src/main/java/org/apache/sqoop/connector/idf/IntermediateDataFormatError.java
4d41679
connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestCSVIntermediateDataFormat.java
fcf6c3c
Diff: https://reviews.apache.org/r/28139/diff/
Testing
-------
yes unit tests added.
Thanks,
Veena Basavaraj
