----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23301/ -----------------------------------------------------------
Review request for oozie. Bugs: OOZIE-1918 https://issues.apache.org/jira/browse/OOZIE-1918 Repository: oozie-git Description ------- Subclasses of ActionXCommand have some duplicated code. Refactoring make it more neat. This patch contain: * Renaming CompletedActionXCommand to ActionCompletedXCommand and extending ActionXCommand * Moving some variables and methods to ActionXCommand and making it protected * More meaningful log message : On exeucte(), actionId is duplicated so removed * Adding handleExecutionFail() to handle ActionExcutionException during execute() Diffs ----- core/src/main/java/org/apache/oozie/DagEngine.java f6e5b4f core/src/main/java/org/apache/oozie/command/wf/ActionCheckXCommand.java 39c522d core/src/main/java/org/apache/oozie/command/wf/ActionCompletedXCommand.java PRE-CREATION core/src/main/java/org/apache/oozie/command/wf/ActionEndXCommand.java 42cb290 core/src/main/java/org/apache/oozie/command/wf/ActionKillXCommand.java 863bf7d core/src/main/java/org/apache/oozie/command/wf/ActionStartXCommand.java 5d90775 core/src/main/java/org/apache/oozie/command/wf/ActionXCommand.java 0c4c548 core/src/main/java/org/apache/oozie/command/wf/CompletedActionXCommand.java 41e6732 Diff: https://reviews.apache.org/r/23301/diff/ Testing ------- No added new testcase. Done manual test. Thanks, Azrael Park