----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17720/ -----------------------------------------------------------
(Updated March 17, 2014, 7:05 p.m.) Review request for oozie. Changes ------- fixed review comments. timeout --> 5 sec is the same with XCommand, discusssed with Mona on needs of requeuing jobevent when it's not processed due to lock, and concluded not needed.(updateAllStatus is able to pick up) test case --> changed coord-action-sla.xml to ensure only 1 coord action created. Bugs: OOZIE-1678 https://issues.apache.org/jira/browse/OOZIE-1678 Repository: oozie-git Description ------- https://issues.apache.org/jira/browse/OOZIE-1678 Diffs (updated) ----- core/src/main/java/org/apache/oozie/event/EventQueue.java 39156d6 core/src/main/java/org/apache/oozie/event/MemoryEventQueue.java f32afb4 core/src/main/java/org/apache/oozie/executor/jpa/SLARegistrationQueryExecutor.java e3b115f core/src/main/java/org/apache/oozie/executor/jpa/SLASummaryQueryExecutor.java 79d11ed core/src/main/java/org/apache/oozie/service/EventHandlerService.java 4207a07 core/src/main/java/org/apache/oozie/service/JPAService.java aba8709 core/src/main/java/org/apache/oozie/sla/SLACalcStatus.java ea53712 core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java 618d899 core/src/main/java/org/apache/oozie/sla/SLARegistrationBean.java a2260a4 core/src/main/java/org/apache/oozie/sla/SLASummaryBean.java 0a70326 core/src/main/java/org/apache/oozie/sla/service/SLAService.java 2458e69 core/src/test/java/org/apache/oozie/executor/jpa/TestSLARegistrationQueryExecutor.java 00fb677 core/src/test/java/org/apache/oozie/executor/jpa/TestSLASummaryQueryExecutor.java 2e170a4 core/src/test/java/org/apache/oozie/service/TestHASLAService.java PRE-CREATION core/src/test/java/org/apache/oozie/sla/TestSLAEventGeneration.java f3bfc29 core/src/test/java/org/apache/oozie/test/ZKXTestCase.java 7bebaf0 core/src/test/resources/coord-action-sla.xml e88df6c Diff: https://reviews.apache.org/r/17720/diff/ Testing ------- Thanks, Ryota Egashira