Sqoop, mail # dev - Review Request: SQOOP-784: Sqoop2: Creating job for connection with id > 1 will results in NullPointerException

Review Request: SQOOP-784: Sqoop2: Creating job for connection with id > 1 will results in NullPointerException

Jarek Cecho 2012-12-17, 18:29

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

Review request for Sqoop.
Description
-------

It seems that we were using connection id instead of connector id.
This addresses bug SQOOP-784.
https://issues.apache.org/jira/browse/SQOOP-784
Diffs
-----

client/src/main/java/org/apache/sqoop/client/shell/CreateJobFunction.java 2b55adc

Diff: https://reviews.apache.org/r/8641/diff/
Testing
-------
Thanks,

Jarek Cecho

Re: Review Request: SQOOP-784: Sqoop2: Creating job for connection with id > 1 will results in NullPointerException

Cheolsoo Park 2012-12-17, 21:26

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

Ship it!
Ship It!

- Cheolsoo Park
On Dec. 17, 2012, 6:29 p.m., Jarek Cecho wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8641/
> -----------------------------------------------------------
>
> (Updated Dec. 17, 2012, 6:29 p.m.)
>
>
> Review request for Sqoop.
>
>
> Description
> -------
>
> It seems that we were using connection id instead of connector id.
>
>
> This addresses bug SQOOP-784.
> https://issues.apache.org/jira/browse/SQOOP-784
>
>
> Diffs
> -----
>
> client/src/main/java/org/apache/sqoop/client/shell/CreateJobFunction.java 2b55adc
>
> Diff: https://reviews.apache.org/r/8641/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Jarek Cecho
>
>