Oozie
  1. Oozie
  2. OOZIE-794

oozie job -info -filter should error out on bundle job and workflow job

    Details

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

      Activity

      Hide
      jiraposter@reviews.apache.org added a comment -

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

      Review request for oozie.

      Summary
      -------

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

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

      Diffs


      trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java 1306959

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

      Testing
      -------

      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/4557/ ----------------------------------------------------------- Review request for oozie. Summary ------- https://issues.apache.org/jira/browse/OOZIE-794 This addresses bug oozie-794. https://issues.apache.org/jira/browse/oozie-794 Diffs trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java 1306959 Diff: https://reviews.apache.org/r/4557/diff Testing ------- 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/4557/#review6532
      -----------------------------------------------------------

      Does this need unit test cases?

      • Santhosh

      On 2012-03-29 16:59:49, Virag Kothari wrote:

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

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

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

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

      (Updated 2012-03-29 16:59:49)

      Review request for oozie.

      Summary

      -------

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

      This addresses bug oozie-794.

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

      Diffs

      -----

      trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java 1306959

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

      Testing

      -------

      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/4557/#review6532 ----------------------------------------------------------- Does this need unit test cases? Santhosh On 2012-03-29 16:59:49, Virag Kothari wrote: ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4557/ ----------------------------------------------------------- (Updated 2012-03-29 16:59:49) Review request for oozie. Summary ------- https://issues.apache.org/jira/browse/OOZIE-794 This addresses bug oozie-794. https://issues.apache.org/jira/browse/oozie-794 Diffs ----- trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java 1306959 Diff: https://reviews.apache.org/r/4557/diff Testing ------- Thanks, Virag
      Hide
      jiraposter@reviews.apache.org added a comment -

      On 2012-03-29 19:25:25, Santhosh Srinivasan wrote:

      > Does this need unit test cases?

      I dont think it is necessary as its a minor change.

      • Virag

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

      On 2012-03-29 16:59:49, Virag Kothari wrote:

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

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

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

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

      (Updated 2012-03-29 16:59:49)

      Review request for oozie.

      Summary

      -------

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

      This addresses bug oozie-794.

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

      Diffs

      -----

      trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java 1306959

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

      Testing

      -------

      Thanks,

      Virag

      Show
      jiraposter@reviews.apache.org added a comment - On 2012-03-29 19:25:25, Santhosh Srinivasan wrote: > Does this need unit test cases? I dont think it is necessary as its a minor change. Virag ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4557/#review6532 ----------------------------------------------------------- On 2012-03-29 16:59:49, Virag Kothari wrote: ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4557/ ----------------------------------------------------------- (Updated 2012-03-29 16:59:49) Review request for oozie. Summary ------- https://issues.apache.org/jira/browse/OOZIE-794 This addresses bug oozie-794. https://issues.apache.org/jira/browse/oozie-794 Diffs ----- trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java 1306959 Diff: https://reviews.apache.org/r/4557/diff Testing ------- 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/4557/#review6638
      -----------------------------------------------------------

      Ship it!

      +1 committing

      • Mohammad

      On 2012-03-29 16:59:49, Virag Kothari wrote:

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

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

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

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

      (Updated 2012-03-29 16:59:49)

      Review request for oozie.

      Summary

      -------

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

      This addresses bug oozie-794.

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

      Diffs

      -----

      trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java 1306959

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

      Testing

      -------

      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/4557/#review6638 ----------------------------------------------------------- Ship it! +1 committing Mohammad On 2012-03-29 16:59:49, Virag Kothari wrote: ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4557/ ----------------------------------------------------------- (Updated 2012-03-29 16:59:49) Review request for oozie. Summary ------- https://issues.apache.org/jira/browse/OOZIE-794 This addresses bug oozie-794. https://issues.apache.org/jira/browse/oozie-794 Diffs ----- trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java 1306959 Diff: https://reviews.apache.org/r/4557/diff Testing ------- Thanks, 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)

        People

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

          Dates

          • Created:
            Updated:
            Resolved:

            Development