Oozie
  1. Oozie
  2. OOZIE-673

offset and len options not working as expected

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.2.0
    • Component/s: None
    • Labels:
      None

      Description

      oozie job -info 0022981-111110182218199-oozie-wrkf-C -offset 3

      ID Status Ext ID Err Code Created
      Nominal Time
      0022981-111110182218199-oozie-wrkf-C@1 SUCCEEDED 0022982-111110182218199-oozie-wrkf-W - 2012-01-16 06:24
      2010-01-01 00:00
      ------------------------------------------------------------------------------------------------------------------------------------
      0022981-111110182218199-oozie-wrkf-C@4 SUCCEEDED 0022985-111110182218199-oozie-wrkf-W - 2012-01-16 06:24
      2010-01-01 00:30
      ------------------------------------------------------------------------------------------------------------------------------------
      0022981-111110182218199-oozie-wrkf-C@5 SUCCEEDED 0022986-111110182218199-oozie-wrkf-W - 2012-01-16 06:24
      2010-01-01 00:40
      ------------------------------------------------------------------------------------------------------------------------------------

      The jira proposes to order actions by nominal time

      Also, the info command may timeout if the number of coordinator actions are very high. So, the default number of actions to retrieve while using the info should be oozie-site configurable.

        Activity

        Virag Kothari created issue -
        Hide
        jiraposter@reviews.apache.org added a comment -

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

        Review request for oozie.

        Summary
        -------

        https://issues.apache.org/jira/browse/OOZIE-673

        This addresses bug oozie-673.
        https://issues.apache.org/jira/browse/oozie-673

        Diffs


        trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737
        trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737
        trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737
        trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737
        trunk/core/src/main/resources/oozie-default.xml 1239737
        trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737

        Diff: https://reviews.apache.org/r/3738/diff

        Testing
        -------

        test for sorting added
        tested after running cron example

        Thanks,

        Virag

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3738/ ----------------------------------------------------------- Review request for oozie. Summary ------- https://issues.apache.org/jira/browse/OOZIE-673 This addresses bug oozie-673. https://issues.apache.org/jira/browse/oozie-673 Diffs trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737 trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737 trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737 trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737 trunk/core/src/main/resources/oozie-default.xml 1239737 trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737 Diff: https://reviews.apache.org/r/3738/diff Testing ------- test for sorting added tested after running cron example Thanks, Virag
        Hide
        jiraposter@reviews.apache.org added a comment -

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

        trunk/core/src/main/resources/oozie-default.xml
        <https://reviews.apache.org/r/3738/#comment10493>

        the default value in oozie-default.xml and in the code should be the same. and 100000 seems a bit too high, shouldn't we cap on 1000 as default?

        • Alejandro

        On 2012-02-02 18:46:30, Virag Kothari wrote:

        -----------------------------------------------------------

        This is an automatically generated e-mail. To reply, visit:

        https://reviews.apache.org/r/3738/

        -----------------------------------------------------------

        (Updated 2012-02-02 18:46:30)

        Review request for oozie.

        Summary

        -------

        https://issues.apache.org/jira/browse/OOZIE-673

        This addresses bug oozie-673.

        https://issues.apache.org/jira/browse/oozie-673

        Diffs

        -----

        trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737

        trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737

        trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737

        trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737

        trunk/core/src/main/resources/oozie-default.xml 1239737

        trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737

        Diff: https://reviews.apache.org/r/3738/diff

        Testing

        -------

        test for sorting added

        tested after running cron example

        Thanks,

        Virag

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3738/#review4771 ----------------------------------------------------------- trunk/core/src/main/resources/oozie-default.xml < https://reviews.apache.org/r/3738/#comment10493 > the default value in oozie-default.xml and in the code should be the same. and 100000 seems a bit too high, shouldn't we cap on 1000 as default? Alejandro On 2012-02-02 18:46:30, Virag Kothari wrote: ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3738/ ----------------------------------------------------------- (Updated 2012-02-02 18:46:30) Review request for oozie. Summary ------- https://issues.apache.org/jira/browse/OOZIE-673 This addresses bug oozie-673. https://issues.apache.org/jira/browse/oozie-673 Diffs ----- trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737 trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737 trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737 trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737 trunk/core/src/main/resources/oozie-default.xml 1239737 trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737 Diff: https://reviews.apache.org/r/3738/diff Testing ------- test for sorting added tested after running cron example Thanks, Virag
        Hide
        jiraposter@reviews.apache.org added a comment -

        On 2012-02-02 18:54:41, Alejandro Abdelnur wrote:

        > trunk/core/src/main/resources/oozie-default.xml, line 1519

        > <https://reviews.apache.org/r/3738/diff/1/?file=71881#file71881line1519>

        >

        > the default value in oozie-default.xml and in the code should be the same. and 100000 seems a bit too high, shouldn't we cap on 1000 as default?

        the default value is kept high to not break the current info api. There are some coord jobs with around 10,000 actions. But, I understand that 100000 is also too high. So, should I make it 10,000?
        Also will reflect that value in the code

        • Virag

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

        On 2012-02-02 18:46:30, Virag Kothari wrote:

        -----------------------------------------------------------

        This is an automatically generated e-mail. To reply, visit:

        https://reviews.apache.org/r/3738/

        -----------------------------------------------------------

        (Updated 2012-02-02 18:46:30)

        Review request for oozie.

        Summary

        -------

        https://issues.apache.org/jira/browse/OOZIE-673

        This addresses bug oozie-673.

        https://issues.apache.org/jira/browse/oozie-673

        Diffs

        -----

        trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737

        trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737

        trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737

        trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737

        trunk/core/src/main/resources/oozie-default.xml 1239737

        trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737

        Diff: https://reviews.apache.org/r/3738/diff

        Testing

        -------

        test for sorting added

        tested after running cron example

        Thanks,

        Virag

        Show
        jiraposter@reviews.apache.org added a comment - On 2012-02-02 18:54:41, Alejandro Abdelnur wrote: > trunk/core/src/main/resources/oozie-default.xml, line 1519 > < https://reviews.apache.org/r/3738/diff/1/?file=71881#file71881line1519 > > > the default value in oozie-default.xml and in the code should be the same. and 100000 seems a bit too high, shouldn't we cap on 1000 as default? the default value is kept high to not break the current info api. There are some coord jobs with around 10,000 actions. But, I understand that 100000 is also too high. So, should I make it 10,000? Also will reflect that value in the code Virag ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3738/#review4771 ----------------------------------------------------------- On 2012-02-02 18:46:30, Virag Kothari wrote: ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3738/ ----------------------------------------------------------- (Updated 2012-02-02 18:46:30) Review request for oozie. Summary ------- https://issues.apache.org/jira/browse/OOZIE-673 This addresses bug oozie-673. https://issues.apache.org/jira/browse/oozie-673 Diffs ----- trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737 trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737 trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737 trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737 trunk/core/src/main/resources/oozie-default.xml 1239737 trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737 Diff: https://reviews.apache.org/r/3738/diff Testing ------- test for sorting added tested after running cron example Thanks, Virag
        Hide
        jiraposter@reviews.apache.org added a comment -

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

        (Updated 2012-02-02 20:00:50.659432)

        Review request for oozie.

        Changes
        -------

        Integrating Alejandro's comments
        Adding info to twiki

        Summary
        -------

        https://issues.apache.org/jira/browse/OOZIE-673

        This addresses bug oozie-673.
        https://issues.apache.org/jira/browse/oozie-673

        Diffs (updated)


        trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737
        trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737
        trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737
        trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737
        trunk/core/src/main/resources/oozie-default.xml 1239737
        trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737
        trunk/docs/src/site/twiki/DG_CommandLineTool.twiki 1239737

        Diff: https://reviews.apache.org/r/3738/diff

        Testing
        -------

        test for sorting added
        tested after running cron example

        Thanks,

        Virag

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3738/ ----------------------------------------------------------- (Updated 2012-02-02 20:00:50.659432) Review request for oozie. Changes ------- Integrating Alejandro's comments Adding info to twiki Summary ------- https://issues.apache.org/jira/browse/OOZIE-673 This addresses bug oozie-673. https://issues.apache.org/jira/browse/oozie-673 Diffs (updated) trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737 trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737 trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737 trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737 trunk/core/src/main/resources/oozie-default.xml 1239737 trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737 trunk/docs/src/site/twiki/DG_CommandLineTool.twiki 1239737 Diff: https://reviews.apache.org/r/3738/diff Testing ------- test for sorting added tested after running cron example Thanks, Virag
        Hide
        jiraposter@reviews.apache.org added a comment -

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

        Looks good.

        A 10000 resultset by default means a large DB result, caching locally in the server (because the JSON structure is built in memory) and then network traffic.

        Still, I'd argue a more sensible default is 1000. I don't see how this will break the client as it is a just a different number of rows being returned.

        • Alejandro

        On 2012-02-02 20:00:50, Virag Kothari wrote:

        -----------------------------------------------------------

        This is an automatically generated e-mail. To reply, visit:

        https://reviews.apache.org/r/3738/

        -----------------------------------------------------------

        (Updated 2012-02-02 20:00:50)

        Review request for oozie.

        Summary

        -------

        https://issues.apache.org/jira/browse/OOZIE-673

        This addresses bug oozie-673.

        https://issues.apache.org/jira/browse/oozie-673

        Diffs

        -----

        trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737

        trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737

        trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737

        trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737

        trunk/core/src/main/resources/oozie-default.xml 1239737

        trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737

        trunk/docs/src/site/twiki/DG_CommandLineTool.twiki 1239737

        Diff: https://reviews.apache.org/r/3738/diff

        Testing

        -------

        test for sorting added

        tested after running cron example

        Thanks,

        Virag

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3738/#review4774 ----------------------------------------------------------- Looks good. A 10000 resultset by default means a large DB result, caching locally in the server (because the JSON structure is built in memory) and then network traffic. Still, I'd argue a more sensible default is 1000. I don't see how this will break the client as it is a just a different number of rows being returned. Alejandro On 2012-02-02 20:00:50, Virag Kothari wrote: ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3738/ ----------------------------------------------------------- (Updated 2012-02-02 20:00:50) Review request for oozie. Summary ------- https://issues.apache.org/jira/browse/OOZIE-673 This addresses bug oozie-673. https://issues.apache.org/jira/browse/oozie-673 Diffs ----- trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737 trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737 trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737 trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737 trunk/core/src/main/resources/oozie-default.xml 1239737 trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737 trunk/docs/src/site/twiki/DG_CommandLineTool.twiki 1239737 Diff: https://reviews.apache.org/r/3738/diff Testing ------- test for sorting added tested after running cron example Thanks, Virag
        Hide
        jiraposter@reviews.apache.org added a comment -

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

        (Updated 2012-02-02 20:59:14.944877)

        Review request for oozie.

        Changes
        -------

        Integrating Alejandro's comment about default value to be 1000.
        Regarding that comment, the info api might break if the client has 10,000 actions. Now they will have to loop 10 times to get all actions. But that can be addressed by overriding this value in oozie-site. I agree its better to have a lower value to avoid high memory and n/w usage.

        Summary
        -------

        https://issues.apache.org/jira/browse/OOZIE-673

        This addresses bug oozie-673.
        https://issues.apache.org/jira/browse/oozie-673

        Diffs (updated)


        trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737
        trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737
        trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737
        trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737
        trunk/core/src/main/resources/oozie-default.xml 1239737
        trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737
        trunk/docs/src/site/twiki/DG_CommandLineTool.twiki 1239737

        Diff: https://reviews.apache.org/r/3738/diff

        Testing
        -------

        test for sorting added
        tested after running cron example

        Thanks,

        Virag

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3738/ ----------------------------------------------------------- (Updated 2012-02-02 20:59:14.944877) Review request for oozie. Changes ------- Integrating Alejandro's comment about default value to be 1000. Regarding that comment, the info api might break if the client has 10,000 actions. Now they will have to loop 10 times to get all actions. But that can be addressed by overriding this value in oozie-site. I agree its better to have a lower value to avoid high memory and n/w usage. Summary ------- https://issues.apache.org/jira/browse/OOZIE-673 This addresses bug oozie-673. https://issues.apache.org/jira/browse/oozie-673 Diffs (updated) trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737 trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737 trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737 trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737 trunk/core/src/main/resources/oozie-default.xml 1239737 trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737 trunk/docs/src/site/twiki/DG_CommandLineTool.twiki 1239737 Diff: https://reviews.apache.org/r/3738/diff Testing ------- test for sorting added tested after running cron example Thanks, Virag
        Hide
        jiraposter@reviews.apache.org added a comment -

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

        Ship it!

        • Alejandro

        On 2012-02-02 20:59:14, Virag Kothari wrote:

        -----------------------------------------------------------

        This is an automatically generated e-mail. To reply, visit:

        https://reviews.apache.org/r/3738/

        -----------------------------------------------------------

        (Updated 2012-02-02 20:59:14)

        Review request for oozie.

        Summary

        -------

        https://issues.apache.org/jira/browse/OOZIE-673

        This addresses bug oozie-673.

        https://issues.apache.org/jira/browse/oozie-673

        Diffs

        -----

        trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737

        trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737

        trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737

        trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737

        trunk/core/src/main/resources/oozie-default.xml 1239737

        trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737

        trunk/docs/src/site/twiki/DG_CommandLineTool.twiki 1239737

        Diff: https://reviews.apache.org/r/3738/diff

        Testing

        -------

        test for sorting added

        tested after running cron example

        Thanks,

        Virag

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3738/#review4775 ----------------------------------------------------------- Ship it! Alejandro On 2012-02-02 20:59:14, Virag Kothari wrote: ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3738/ ----------------------------------------------------------- (Updated 2012-02-02 20:59:14) Review request for oozie. Summary ------- https://issues.apache.org/jira/browse/OOZIE-673 This addresses bug oozie-673. https://issues.apache.org/jira/browse/oozie-673 Diffs ----- trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737 trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737 trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737 trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737 trunk/core/src/main/resources/oozie-default.xml 1239737 trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737 trunk/docs/src/site/twiki/DG_CommandLineTool.twiki 1239737 Diff: https://reviews.apache.org/r/3738/diff Testing ------- test for sorting added tested after running cron example Thanks, Virag
        Hide
        jiraposter@reviews.apache.org added a comment -

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

        Ship it!

        committing and will close

        • Mohammad

        On 2012-02-02 20:59:14, Virag Kothari wrote:

        -----------------------------------------------------------

        This is an automatically generated e-mail. To reply, visit:

        https://reviews.apache.org/r/3738/

        -----------------------------------------------------------

        (Updated 2012-02-02 20:59:14)

        Review request for oozie.

        Summary

        -------

        https://issues.apache.org/jira/browse/OOZIE-673

        This addresses bug oozie-673.

        https://issues.apache.org/jira/browse/oozie-673

        Diffs

        -----

        trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737

        trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737

        trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737

        trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737

        trunk/core/src/main/resources/oozie-default.xml 1239737

        trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737

        trunk/docs/src/site/twiki/DG_CommandLineTool.twiki 1239737

        Diff: https://reviews.apache.org/r/3738/diff

        Testing

        -------

        test for sorting added

        tested after running cron example

        Thanks,

        Virag

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3738/#review4787 ----------------------------------------------------------- Ship it! committing and will close Mohammad On 2012-02-02 20:59:14, Virag Kothari wrote: ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3738/ ----------------------------------------------------------- (Updated 2012-02-02 20:59:14) Review request for oozie. Summary ------- https://issues.apache.org/jira/browse/OOZIE-673 This addresses bug oozie-673. https://issues.apache.org/jira/browse/oozie-673 Diffs ----- trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1239737 trunk/core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java 1239737 trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 1239737 trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1239737 trunk/core/src/main/resources/oozie-default.xml 1239737 trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java 1239737 trunk/docs/src/site/twiki/DG_CommandLineTool.twiki 1239737 Diff: https://reviews.apache.org/r/3738/diff Testing ------- test for sorting added tested after running cron example Thanks, Virag
        Virag Kothari made changes -
        Field Original Value New Value
        Status Open [ 1 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        Alejandro Abdelnur made changes -
        Resolution Fixed [ 1 ]
        Status Closed [ 6 ] Reopened [ 4 ]
        Alejandro Abdelnur made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Alejandro Abdelnur made changes -
        Fix Version/s 3.2.0 [ 12320149 ]
        Gavin made changes -
        Workflow jira [ 12651686 ] no-reopen-closed, patch-avail [ 12655864 ]
        Alejandro Abdelnur made changes -
        Assignee Virag Kothari [ virag ]
        Hide
        Virag Kothari added a comment -

        For the patch committed in trunk with this JIRA ID (also available under the same JIRA ID in Apache ReviewBoard system), I grant license to ASF for inclusion in ASF works (as per the Apache License §5)

        Show
        Virag Kothari added a comment - For the patch committed in trunk with this JIRA ID (also available under the same JIRA ID in Apache ReviewBoard system), I grant license to ASF for inclusion in ASF works (as per the Apache License §5)
        Alejandro Abdelnur made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Virag Kothari
            Reporter:
            Virag Kothari
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development