> On Dec. 1, 2014, 4:55 p.m., Jarek Cecho wrote:
> > connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestCSVIntermediateDataFormat.java,
line 214
> > <https://reviews.apache.org/r/28583/diff/1/?file=779693#file779693line214>
> >
> > I believe that the size have to be increased to 7 here as I'm getting:
> >
> > java.lang.ArrayIndexOutOfBoundsException: 6
> > at org.apache.sqoop.connector.idf.TestCSVIntermediateDataFormat.testObjectArrayWithNullInCSVTextOut(TestCSVIntermediateDataFormat.java:221)
> > ...
fixed, updated patch. last minute CLTR+Z !
- Veena
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28583/#review63474
-----------------------------------------------------------
On Dec. 1, 2014, 3:55 p.m., Veena Basavaraj wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/28583/
> -----------------------------------------------------------
>
> (Updated Dec. 1, 2014, 3:55 p.m.)
>
>
> Review request for Sqoop.
>
>
> Bugs: SQOOP-1819
> https://issues.apache.org/jira/browse/SQOOP-1819
>
>
> Repository: sqoop-sqoop2
>
>
> Description
> -------
>
> see JIRA
>
>
> Diffs
> -----
>
> connector/connector-sdk/src/main/java/org/apache/sqoop/connector/idf/CSVIntermediateDataFormat.java
bdab7a4
> connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestCSVIntermediateDataFormat.java
bd082aa
>
> Diff: https://reviews.apache.org/r/28583/diff/
>
>
> Testing
> -------
>
> yes
>
>
> Thanks,
>
> Veena Basavaraj
>
>
|