----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6311/#review9974 -----------------------------------------------------------
trunk/core/src/main/java/org/apache/oozie/workflow/lite/LiteWorkflowAppParser.java <https://reviews.apache.org/r/6311/#comment21166> NPE shouldn't be caught. Can you instead do a check for null? - Virag Kothari On Aug. 7, 2012, 5:17 p.m., Britt Cyr wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/6311/ > ----------------------------------------------------------- > > (Updated Aug. 7, 2012, 5:17 p.m.) > > > Review request for oozie. > > > Description > ------- > > OOZIE-921 Changes in global section for the Name Node in FS action > > > Diffs > ----- > > trunk/core/src/main/java/org/apache/oozie/action/ActionExecutor.java > 1370356 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java > 1370356 > > trunk/core/src/main/java/org/apache/oozie/workflow/lite/LiteWorkflowAppParser.java > 1370356 > > trunk/core/src/test/java/org/apache/oozie/workflow/lite/TestLiteWorkflowAppParser.java > 1370356 > > Diff: https://reviews.apache.org/r/6311/diff/ > > > Testing > ------- > > > Thanks, > > Britt Cyr > >
