You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sqoop.apache.org by Bilung Lee <bl...@gmail.com> on 2012/11/15 03:10:06 UTC
Review Request: SQOOP-693 Intermediate data format support for export
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8064/
-----------------------------------------------------------
Review request for Sqoop.
Description
-------
Provide conversion from CSV record to array record.
This addresses bug SQOOP-693.
https://issues.apache.org/jira/browse/SQOOP-693
Diffs
-----
execution/mapreduce/src/main/java/org/apache/sqoop/job/io/Data.java f6fff0b
execution/mapreduce/src/test/java/org/apache/sqoop/job/io/TestData.java ea7ac70
Diff: https://reviews.apache.org/r/8064/diff/
Testing
-------
Thanks,
Bilung Lee
Re: Review Request: SQOOP-693 Intermediate data format support for export
Posted by Bilung Lee <bl...@gmail.com>.
> On Nov. 15, 2012, 11:20 p.m., Jarek Cecho wrote:
> > Thank you for your time Bilung. Changes looks good to me, please attach your patch to the JIRA and I'll commit it.
Thanks for your review. Patch is attached to JIRA now.
- Bilung
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8064/#review13500
-----------------------------------------------------------
On Nov. 15, 2012, 11:12 p.m., Bilung Lee wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8064/
> -----------------------------------------------------------
>
> (Updated Nov. 15, 2012, 11:12 p.m.)
>
>
> Review request for Sqoop.
>
>
> Description
> -------
>
> Provide conversion from CSV record to array record.
>
>
> This addresses bug SQOOP-693.
> https://issues.apache.org/jira/browse/SQOOP-693
>
>
> Diffs
> -----
>
> execution/mapreduce/src/main/java/org/apache/sqoop/job/io/Data.java f6fff0b
> execution/mapreduce/src/test/java/org/apache/sqoop/job/io/TestData.java ea7ac70
>
> Diff: https://reviews.apache.org/r/8064/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Bilung Lee
>
>
Re: Review Request: SQOOP-693 Intermediate data format support for export
Posted by Jarek Cecho <ja...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8064/#review13500
-----------------------------------------------------------
Ship it!
Thank you for your time Bilung. Changes looks good to me, please attach your patch to the JIRA and I'll commit it.
- Jarek Cecho
On Nov. 15, 2012, 11:12 p.m., Bilung Lee wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8064/
> -----------------------------------------------------------
>
> (Updated Nov. 15, 2012, 11:12 p.m.)
>
>
> Review request for Sqoop.
>
>
> Description
> -------
>
> Provide conversion from CSV record to array record.
>
>
> This addresses bug SQOOP-693.
> https://issues.apache.org/jira/browse/SQOOP-693
>
>
> Diffs
> -----
>
> execution/mapreduce/src/main/java/org/apache/sqoop/job/io/Data.java f6fff0b
> execution/mapreduce/src/test/java/org/apache/sqoop/job/io/TestData.java ea7ac70
>
> Diff: https://reviews.apache.org/r/8064/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Bilung Lee
>
>
Re: Review Request: SQOOP-693 Intermediate data format support for export
Posted by Bilung Lee <bl...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8064/
-----------------------------------------------------------
(Updated Nov. 15, 2012, 11:12 p.m.)
Review request for Sqoop.
Changes
-------
repatch to base on current code.
Description
-------
Provide conversion from CSV record to array record.
This addresses bug SQOOP-693.
https://issues.apache.org/jira/browse/SQOOP-693
Diffs (updated)
-----
execution/mapreduce/src/main/java/org/apache/sqoop/job/io/Data.java f6fff0b
execution/mapreduce/src/test/java/org/apache/sqoop/job/io/TestData.java ea7ac70
Diff: https://reviews.apache.org/r/8064/diff/
Testing
-------
Thanks,
Bilung Lee