----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57870/ -----------------------------------------------------------
(Updated 四月 1, 2017, 11:31 a.m.) Review request for oozie. Changes ------- 1 add comment to the ambiguous method with empty body 2 Use empty WorkflowActionBean instead of null as placeholder element 3 Make WorkflowActionGetJPAExecutor be able to handle both "acceptable null" and "null exception" Repository: oozie-git Description ------- More detailed view of the coordinator’s history can be observed from perspective of workflow action. Diffs (updated) ----- client/src/main/java/org/apache/oozie/client/rest/RestConstants.java 8ddb1f8 core/src/main/java/org/apache/oozie/CoordinatorEngine.java 2f9f822 core/src/main/java/org/apache/oozie/ErrorCode.java b03ad06 core/src/main/java/org/apache/oozie/command/coord/CoordWfActionInfoXCommand.java PRE-CREATION core/src/main/java/org/apache/oozie/executor/jpa/WorkflowActionGetJPAExecutor.java 0b7f50d core/src/main/java/org/apache/oozie/servlet/BaseJobServlet.java 03acbc1 core/src/main/java/org/apache/oozie/servlet/V2JobServlet.java 6c30f5d core/src/test/java/org/apache/oozie/client/TestOozieCLI.java 564db2a core/src/test/java/org/apache/oozie/command/coord/TestCoordWfActionInfoXCommand.java PRE-CREATION core/src/test/java/org/apache/oozie/servlet/MockCoordinatorEngineService.java 4fc8653 core/src/test/java/org/apache/oozie/servlet/TestV2JobServlet.java fb203a6 Diff: https://reviews.apache.org/r/57870/diff/4/ Changes: https://reviews.apache.org/r/57870/diff/3-4/ Testing ------- Thanks, Alonzo Zhou
