----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6876/#review10951 -----------------------------------------------------------
trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleKillXCommand.java <https://reviews.apache.org/r/6876/#comment23509> Why are the bundle job status values equated with CoordinatorJob.Status? Status enum is declared in parent class Job itself so Job.Status can be used here like before to avoid confusion trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleKillXCommand.java <https://reviews.apache.org/r/6876/#comment23510> If status was KILLED, how does the log say "not killed"? I may be missing something here - Mona Chitnis On Aug. 31, 2012, 6:21 p.m., Virag Kothari wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/6876/ > ----------------------------------------------------------- > > (Updated Aug. 31, 2012, 6:21 p.m.) > > > Review request for oozie. > > > Description > ------- > > https://issues.apache.org/jira/browse/OOZIE-978 > > > This addresses bug OOZIE-978. > https://issues.apache.org/jira/browse/OOZIE-978 > > > Diffs > ----- > > > trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleJobSuspendXCommand.java > 1379483 > > trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleKillXCommand.java > 1379483 > > trunk/core/src/test/java/org/apache/oozie/service/TestStatusTransitService.java > 1379483 > > Diff: https://reviews.apache.org/r/6876/diff/ > > > Testing > ------- > > > Thanks, > > Virag Kothari > >
