Re: Review Request 17720: OOZIE-1678 HA support for SLA

[Ryota Egashira]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17720/
-----------------------------------------------------------

(Updated May 7, 2014, 4:46 a.m.)


Review request for oozie.


Changes
-------

fixed review comments


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/command/coord/CoordChangeXCommand.java 
fb31e9a 
  
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/executor/jpa/sla/SLASummaryGetJPAExecutor.java
 1f7cb4d 
  core/src/main/java/org/apache/oozie/service/JPAService.java aba8709 
  core/src/main/java/org/apache/oozie/service/ZKJobsConcurrencyService.java 
611b74c 
  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 ea2983f 
  
core/src/test/java/org/apache/oozie/command/coord/TestCoordChangeXCommand.java 
327ec90 
  
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/TestSLACalculationJPAExecutor.java 
9270aa2 
  core/src/test/java/org/apache/oozie/sla/TestSLACalculatorMemory.java 9a16722 
  core/src/test/java/org/apache/oozie/sla/TestSLAEventGeneration.java f3bfc29 
  core/src/test/java/org/apache/oozie/sla/TestSLAJobEventListener.java 3ce86ab 
  core/src/test/java/org/apache/oozie/sla/TestSLAService.java 291d850 
  
core/src/test/java/org/apache/oozie/sla/TestSLASummaryGetOnRestartJPAExecutor.java
 3a9e72e 
  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