----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3883/#review5065 -----------------------------------------------------------
trunk/core/src/main/java/org/apache/oozie/workflow/lite/LiteWorkflowAppParser.java <https://reviews.apache.org/r/3883/#comment11112> k trunk/core/src/test/java/org/apache/oozie/workflow/lite/TestLiteWorkflowAppParser.java <https://reviews.apache.org/r/3883/#comment11113> k trunk/core/src/test/java/org/apache/oozie/workflow/lite/TestLiteWorkflowAppParser.java <https://reviews.apache.org/r/3883/#comment11114> k trunk/core/src/test/java/org/apache/oozie/workflow/lite/TestLiteWorkflowAppParser.java <https://reviews.apache.org/r/3883/#comment11115> k trunk/core/src/test/java/org/apache/oozie/workflow/lite/TestLiteWorkflowAppParser.java <https://reviews.apache.org/r/3883/#comment11116> k trunk/core/src/test/java/org/apache/oozie/workflow/lite/TestLiteWorkflowAppParser.java <https://reviews.apache.org/r/3883/#comment11117> k trunk/core/src/test/java/org/apache/oozie/workflow/lite/TestLiteWorkflowAppParser.java <https://reviews.apache.org/r/3883/#comment11119> Yes.. We are more liberal in error-checking for 'error-to' transition of nodes. E.g. Forked actions can directly transition to kill node on 'error-to'. No need of a join there. Also, fork can have multiple edges from a decision node. trunk/core/src/test/java/org/apache/oozie/workflow/lite/TestLiteWorkflowAppParser.java <https://reviews.apache.org/r/3883/#comment11118> k - Virag On 2012-02-13 19:10:15, Virag Kothari wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3883/ > ----------------------------------------------------------- > > (Updated 2012-02-13 19:10:15) > > > Review request for oozie and Santhosh Srinivasan. > > > Summary > ------- > > https://issues.apache.org/jira/browse/OOZIE-693 > > > This addresses bug OOZIE-693. > https://issues.apache.org/jira/browse/OOZIE-693 > > > Diffs > ----- > > trunk/core/src/main/java/org/apache/oozie/ErrorCode.java 1243647 > > trunk/core/src/main/java/org/apache/oozie/workflow/lite/LiteWorkflowAppParser.java > 1243647 > > trunk/core/src/test/java/org/apache/oozie/workflow/lite/TestLiteWorkflowAppParser.java > 1243647 > > Diff: https://reviews.apache.org/r/3883/diff > > > Testing > ------- > > Few tests added > > > Thanks, > > Virag > >
