-----------------------------------------------------------
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

Reply via email to