----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11797/ -----------------------------------------------------------
(Updated June 18, 2013, 9:14 p.m.) Review request for oozie. Changes ------- Made small change in confirm against DB logic, to check job status too, in addition to end-time. Removed other bug fix code from this patch. Please dont look at diff from previous versions, as that view is broken and shows additions/deletions inverted. Look at diff4 as a whole. Description ------- https://issues.apache.org/jira/browse/OOZIE-1379 This addresses bug OOZIE-1379. https://issues.apache.org/jira/browse/OOZIE-1379 Diffs (updated) ----- trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 1494286 trunk/core/src/main/java/org/apache/oozie/WorkflowActionBean.java 1494286 trunk/core/src/main/java/org/apache/oozie/event/BundleJobEvent.java 1494286 trunk/core/src/main/java/org/apache/oozie/event/CoordinatorActionEvent.java 1494286 trunk/core/src/main/java/org/apache/oozie/event/CoordinatorJobEvent.java 1494286 trunk/core/src/main/java/org/apache/oozie/event/WorkflowActionEvent.java 1494286 trunk/core/src/main/java/org/apache/oozie/event/WorkflowJobEvent.java 1494286 trunk/core/src/main/java/org/apache/oozie/executor/jpa/sla/SLASummaryUpdateForSLAStatusActualTimesJPAExecutor.java 1494286 trunk/core/src/main/java/org/apache/oozie/executor/jpa/sla/SLASummaryUpdateForSLAStatusJPAExecutor.java 1494286 trunk/core/src/main/java/org/apache/oozie/sla/SLACalcStatus.java 1494286 trunk/core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java 1494286 trunk/core/src/main/java/org/apache/oozie/sla/SLAOperations.java 1494286 trunk/core/src/main/java/org/apache/oozie/sla/SLARegistrationBean.java 1494286 trunk/core/src/main/java/org/apache/oozie/sla/SLASummaryBean.java 1494286 trunk/core/src/test/java/org/apache/oozie/servlet/TestV2SLAServlet.java 1494286 trunk/core/src/test/java/org/apache/oozie/sla/TestSLACalculationJPAExecutor.java 1494286 trunk/core/src/test/java/org/apache/oozie/sla/TestSLACalculatorMemory.java 1494286 trunk/core/src/test/java/org/apache/oozie/sla/TestSLAJobEventListener.java 1494286 trunk/core/src/test/java/org/apache/oozie/sla/TestSLARegistrationGetJPAExecutor.java 1494286 trunk/core/src/test/java/org/apache/oozie/sla/TestSLARegistrationGetRecordsOnRestartJPAExecutor.java 1494286 trunk/core/src/test/java/org/apache/oozie/sla/TestSLAService.java 1494286 trunk/core/src/test/java/org/apache/oozie/sla/TestSLASummaryGetOnRestartJPAExecutor.java 1494286 trunk/core/src/test/resources/wf-job-sla.xml 1494286 Diff: https://reviews.apache.org/r/11797/diff/ Testing ------- added unit test Thanks, Mona Chitnis
