You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@oozie.apache.org by Narayan Periwal <na...@inmobi.com> on 2015/09/01 08:16:15 UTC
Review Request 37991: Method getCoordJobInfo(String jobId,
String filter,
int offset, int length, boolean desc) is not present in LocalOozieClientCoord
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37991/
-----------------------------------------------------------
Review request for oozie.
Repository: oozie-git
Description
-------
In LocalOozieClientCoord, method getCoordJobInfo(String jobId) is present, but the corresponding overloaded method getCoordJobInfo(String jobId, String filter, int offset, int length, boolean desc) is not present. Adding this method in LocalOozieClientCoord will enable us to use this in the unit testcases.
Diffs
-----
core/src/main/java/org/apache/oozie/LocalOozieClientCoord.java cf60c37
Diff: https://reviews.apache.org/r/37991/diff/
Testing
-------
Done.
Thanks,
Narayan Periwal
Re: Review Request 37991: Method getCoordJobInfo(String jobId,
String filter,
int offset, int length, boolean desc) is not present in LocalOozieClientCoord
Posted by Jaydeep Vishwakarma <ja...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37991/#review99738
-----------------------------------------------------------
Ship it!
Ship It!
- Jaydeep Vishwakarma
On Sept. 1, 2015, 6:16 a.m., Narayan Periwal wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37991/
> -----------------------------------------------------------
>
> (Updated Sept. 1, 2015, 6:16 a.m.)
>
>
> Review request for oozie.
>
>
> Repository: oozie-git
>
>
> Description
> -------
>
> In LocalOozieClientCoord, method getCoordJobInfo(String jobId) is present, but the corresponding overloaded method getCoordJobInfo(String jobId, String filter, int offset, int length, boolean desc) is not present. Adding this method in LocalOozieClientCoord will enable us to use this in the unit testcases.
>
>
> Diffs
> -----
>
> core/src/main/java/org/apache/oozie/LocalOozieClientCoord.java cf60c37
>
> Diff: https://reviews.apache.org/r/37991/diff/
>
>
> Testing
> -------
>
> Done.
>
>
> Thanks,
>
> Narayan Periwal
>
>