You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@oozie.apache.org by Jaydeep Vishwakarma <ja...@gmail.com> on 2016/07/22 10:35:36 UTC
Re: Review Request 48470: OOZIE-2520 SortBy filter for ordering the
jobs query results
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48470/#review143209
-----------------------------------------------------------
core/src/main/java/org/apache/oozie/store/StoreStatusFilter.java (line 259)
<https://reviews.apache.org/r/48470/#comment209001>
remove extra bracket
core/src/test/java/org/apache/oozie/executor/jpa/TestWorkflowsJobGetJPAExecutor.java (line 21)
<https://reviews.apache.org/r/48470/#comment209002>
import only specific class
- Jaydeep Vishwakarma
On June 9, 2016, 6:02 a.m., Abhishek Bafna wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48470/
> -----------------------------------------------------------
>
> (Updated June 9, 2016, 6:02 a.m.)
>
>
> Review request for oozie.
>
>
> Bugs: OOZIE-2520
> https://issues.apache.org/jira/browse/OOZIE-2520
>
>
> Repository: oozie-git
>
>
> Description
> -------
>
> SortBy filter for ordering the jobs query results
>
>
> Diffs
> -----
>
> client/src/main/java/org/apache/oozie/cli/OozieCLI.java 01efb48
> client/src/main/java/org/apache/oozie/client/OozieClient.java 2ffbd6a
> core/src/main/java/org/apache/oozie/CoordinatorEngine.java 79d0d97
> core/src/main/java/org/apache/oozie/DagEngine.java fd83360
> core/src/main/java/org/apache/oozie/executor/jpa/BundleJobInfoGetJPAExecutor.java 4b631c1
> core/src/main/java/org/apache/oozie/executor/jpa/BundleJobQueryExecutor.java bfd1dae
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobInfoGetJPAExecutor.java c924f44
> core/src/main/java/org/apache/oozie/executor/jpa/WorkflowsJobGetJPAExecutor.java 00d2263
> core/src/main/java/org/apache/oozie/store/StoreStatusFilter.java d442aa8
> core/src/main/java/org/apache/oozie/util/JobsFilterUtils.java 52d3e9d
> core/src/test/java/org/apache/oozie/client/TestOozieCLI.java 54bfc16
> core/src/test/java/org/apache/oozie/executor/jpa/TestBundleJobInfoGetJPAExecutor.java 5714040
> core/src/test/java/org/apache/oozie/executor/jpa/TestBundleJobQueryExecutor.java 97cbb7f
> core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobInfoGetJPAExecutor.java 5335213
> core/src/test/java/org/apache/oozie/executor/jpa/TestWorkflowsJobGetJPAExecutor.java 46b0d05
> docs/src/site/twiki/DG_CommandLineTool.twiki ff1cce5
> docs/src/site/twiki/WebServicesAPI.twiki a34f2d3
>
> Diff: https://reviews.apache.org/r/48470/diff/
>
>
> Testing
> -------
>
> Unit tests and manual testing done.
>
>
> Thanks,
>
> Abhishek Bafna
>
>
Re: Review Request 48470: OOZIE-2520 SortBy filter for ordering the
jobs query results
Posted by Abhishek Bafna <ba...@gmail.com>.
> On July 22, 2016, 10:35 a.m., Jaydeep Vishwakarma wrote:
> > core/src/main/java/org/apache/oozie/store/StoreStatusFilter.java, line 259
> > <https://reviews.apache.org/r/48470/diff/1/?file=1411989#file1411989line259>
> >
> > remove extra bracket
Done.
> On July 22, 2016, 10:35 a.m., Jaydeep Vishwakarma wrote:
> > core/src/test/java/org/apache/oozie/executor/jpa/TestWorkflowsJobGetJPAExecutor.java, line 21
> > <https://reviews.apache.org/r/48470/diff/1/?file=1411995#file1411995line21>
> >
> > import only specific class
Done.
- Abhishek
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48470/#review143209
-----------------------------------------------------------
On June 9, 2016, 6:02 a.m., Abhishek Bafna wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48470/
> -----------------------------------------------------------
>
> (Updated June 9, 2016, 6:02 a.m.)
>
>
> Review request for oozie.
>
>
> Bugs: OOZIE-2520
> https://issues.apache.org/jira/browse/OOZIE-2520
>
>
> Repository: oozie-git
>
>
> Description
> -------
>
> SortBy filter for ordering the jobs query results
>
>
> Diffs
> -----
>
> client/src/main/java/org/apache/oozie/cli/OozieCLI.java 01efb48
> client/src/main/java/org/apache/oozie/client/OozieClient.java 2ffbd6a
> core/src/main/java/org/apache/oozie/CoordinatorEngine.java 79d0d97
> core/src/main/java/org/apache/oozie/DagEngine.java fd83360
> core/src/main/java/org/apache/oozie/executor/jpa/BundleJobInfoGetJPAExecutor.java 4b631c1
> core/src/main/java/org/apache/oozie/executor/jpa/BundleJobQueryExecutor.java bfd1dae
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobInfoGetJPAExecutor.java c924f44
> core/src/main/java/org/apache/oozie/executor/jpa/WorkflowsJobGetJPAExecutor.java 00d2263
> core/src/main/java/org/apache/oozie/store/StoreStatusFilter.java d442aa8
> core/src/main/java/org/apache/oozie/util/JobsFilterUtils.java 52d3e9d
> core/src/test/java/org/apache/oozie/client/TestOozieCLI.java 54bfc16
> core/src/test/java/org/apache/oozie/executor/jpa/TestBundleJobInfoGetJPAExecutor.java 5714040
> core/src/test/java/org/apache/oozie/executor/jpa/TestBundleJobQueryExecutor.java 97cbb7f
> core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobInfoGetJPAExecutor.java 5335213
> core/src/test/java/org/apache/oozie/executor/jpa/TestWorkflowsJobGetJPAExecutor.java 46b0d05
> docs/src/site/twiki/DG_CommandLineTool.twiki ff1cce5
> docs/src/site/twiki/WebServicesAPI.twiki a34f2d3
>
> Diff: https://reviews.apache.org/r/48470/diff/
>
>
> Testing
> -------
>
> Unit tests and manual testing done.
>
>
> Thanks,
>
> Abhishek Bafna
>
>