You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@oozie.apache.org by Rohini Palaniswamy <ro...@gmail.com> on 2013/03/21 20:08:28 UTC
Review Request: [OOZIE-1280] CoordPushDependencyCheck queued by Recovery
Services doesn't remove dependencies from cache
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10065/
-----------------------------------------------------------
Review request for oozie.
Description
-------
Removed available dependencies from cache when CoordPushDependencyCheck is run by RecoveryService.
This addresses bug OOZIE-1280.
https://issues.apache.org/jira/browse/OOZIE-1280
Diffs
-----
http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/command/coord/CoordPushDependencyCheckXCommand.java 1459430
http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/service/RecoveryService.java 1459430
http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordPushDependencyCheckXCommand.java 1459430
http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/service/TestRecoveryService.java 1459430
Diff: https://reviews.apache.org/r/10065/diff/
Testing
-------
Unit test added
Thanks,
Rohini Palaniswamy
Re: Review Request: [OOZIE-1280] CoordPushDependencyCheck queued by Recovery
Services doesn't remove dependencies from cache
Posted by Virag Kothari <vi...@yahoo-inc.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10065/#review18226
-----------------------------------------------------------
Ship it!
Ship It!
- Virag Kothari
On March 21, 2013, 7:19 p.m., Rohini Palaniswamy wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10065/
> -----------------------------------------------------------
>
> (Updated March 21, 2013, 7:19 p.m.)
>
>
> Review request for oozie.
>
>
> Description
> -------
>
> Removed available dependencies from cache when CoordPushDependencyCheck is run by RecoveryService.
>
>
> This addresses bug OOZIE-1280.
> https://issues.apache.org/jira/browse/OOZIE-1280
>
>
> Diffs
> -----
>
> http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/command/coord/CoordPushDependencyCheckXCommand.java 1459430
> http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/service/RecoveryService.java 1459430
> http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordPushDependencyCheckXCommand.java 1459430
> http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/service/TestRecoveryService.java 1459430
>
> Diff: https://reviews.apache.org/r/10065/diff/
>
>
> Testing
> -------
>
> Unit test added
>
>
> Thanks,
>
> Rohini Palaniswamy
>
>
Re: Review Request: [OOZIE-1280] CoordPushDependencyCheck queued by Recovery
Services doesn't remove dependencies from cache
Posted by Rohini Palaniswamy <ro...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10065/
-----------------------------------------------------------
(Updated March 21, 2013, 7:19 p.m.)
Review request for oozie.
Changes
-------
Fixed test failure in TestCoordPushDependencyCheckCommand.java
Description
-------
Removed available dependencies from cache when CoordPushDependencyCheck is run by RecoveryService.
This addresses bug OOZIE-1280.
https://issues.apache.org/jira/browse/OOZIE-1280
Diffs (updated)
-----
http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/command/coord/CoordPushDependencyCheckXCommand.java 1459430
http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/service/RecoveryService.java 1459430
http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordPushDependencyCheckXCommand.java 1459430
http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/service/TestRecoveryService.java 1459430
Diff: https://reviews.apache.org/r/10065/diff/
Testing
-------
Unit test added
Thanks,
Rohini Palaniswamy