You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sqoop.apache.org by Jarek Cecho <ja...@apache.org> on 2012/11/21 06:22:33 UTC
Review Request: SQOOP-673 Mapreduce submission manager should not call
update() on finished submissions
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8154/
-----------------------------------------------------------
Review request for Sqoop.
Description
-------
Very simple patch that is simply putting update() call in status() method into condition and is being executed only in case that loading submission status is considered running.
This addresses bug SQOOP-673.
https://issues.apache.org/jira/browse/SQOOP-673
Diffs
-----
core/src/main/java/org/apache/sqoop/framework/FrameworkManager.java c9c9cf4
Diff: https://reviews.apache.org/r/8154/diff/
Testing
-------
I've tested it on real cluster.
Thanks,
Jarek Cecho
Re: Review Request: SQOOP-673 Mapreduce submission manager should not call
update() on finished submissions
Posted by Bilung Lee <bl...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8154/#review13732
-----------------------------------------------------------
Ship it!
Ship It!
- Bilung Lee
On Nov. 21, 2012, 5:22 a.m., Jarek Cecho wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8154/
> -----------------------------------------------------------
>
> (Updated Nov. 21, 2012, 5:22 a.m.)
>
>
> Review request for Sqoop.
>
>
> Description
> -------
>
> Very simple patch that is simply putting update() call in status() method into condition and is being executed only in case that loading submission status is considered running.
>
>
> This addresses bug SQOOP-673.
> https://issues.apache.org/jira/browse/SQOOP-673
>
>
> Diffs
> -----
>
> core/src/main/java/org/apache/sqoop/framework/FrameworkManager.java c9c9cf4
>
> Diff: https://reviews.apache.org/r/8154/diff/
>
>
> Testing
> -------
>
> I've tested it on real cluster.
>
>
> Thanks,
>
> Jarek Cecho
>
>