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

(Updated April 26, 2013, 9:54 a.m.)


Review request for oozie.


Changes
-------

Patch complete. Includes review comments by Rohini on OOZIE-1209 Job 
Notifications

only TestSLAEventGeneration WIP, and possibly a test to validate new sla schema 
xsd 0.2


Description
-------

1. Revisiting the SLA handling in Oozie
2. Addition of a calculator service to process sla events in a continuous 
fashion
3. Added new oozie-sla schema v0.2, concise and relevant


This addresses bug OOZIE-1244.
    https://issues.apache.org/jira/browse/OOZIE-1244


Diffs (updated)
-----

  trunk/client/src/main/java/org/apache/oozie/AppType.java PRE-CREATION 
  trunk/client/src/main/java/org/apache/oozie/client/SLAEvent.java 1476119 
  trunk/client/src/main/java/org/apache/oozie/client/event/Event.java 1476119 
  trunk/client/src/main/java/org/apache/oozie/client/event/JobEvent.java 
1476119 
  trunk/client/src/main/java/org/apache/oozie/client/event/SLAEvent.java 
1476119 
  
trunk/client/src/main/java/org/apache/oozie/client/event/jms/JSONMessageDeserializer.java
 1476119 
  
trunk/client/src/main/java/org/apache/oozie/client/event/jms/MessageDeserializer.java
 1476119 
  
trunk/client/src/main/java/org/apache/oozie/client/event/message/CoordinatorActionMessage.java
 1476119 
  
trunk/client/src/main/java/org/apache/oozie/client/event/message/EventMessage.java
 1476119 
  
trunk/client/src/main/java/org/apache/oozie/client/event/message/JobMessage.java
 1476119 
  
trunk/client/src/main/java/org/apache/oozie/client/event/message/WorkflowJobMessage.java
 1476119 
  trunk/client/src/main/resources/oozie-sla-0.2.xsd PRE-CREATION 
  trunk/core/pom.xml 1476119 
  trunk/core/src/main/java/org/apache/oozie/ErrorCode.java 1476119 
  trunk/core/src/main/java/org/apache/oozie/SLAEventBean.java 1476119 
  trunk/core/src/main/java/org/apache/oozie/client/rest/JsonSLAEvent.java 
1476119 
  
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionCheckXCommand.java
 1476119 
  
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionInputCheckXCommand.java
 1476119 
  
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionMaterializeCommand.java
 1476119 
  
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionStartXCommand.java
 1476119 
  
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionTimeOutXCommand.java
 1476119 
  
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionUpdateXCommand.java
 1476119 
  
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordMaterializeTransitionXCommand.java
 1476119 
  
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordPushDependencyCheckXCommand.java
 1476119 
  
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordRerunXCommand.java 
1476119 
  
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordinatorXCommand.java
 1476119 
  
trunk/core/src/main/java/org/apache/oozie/command/coord/SLAEventsXCommand.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/command/wf/ActionCheckXCommand.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/command/wf/ActionEndXCommand.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/command/wf/ActionKillXCommand.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/command/wf/ActionStartXCommand.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/command/wf/ActionXCommand.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/command/wf/KillXCommand.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/command/wf/ResumeXCommand.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/command/wf/SignalXCommand.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/command/wf/SubmitXCommand.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/command/wf/SuspendXCommand.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/command/wf/WorkflowXCommand.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/event/BundleJobEvent.java 1476119 
  trunk/core/src/main/java/org/apache/oozie/event/CoordinatorActionEvent.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/event/CoordinatorJobEvent.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/event/EventQueue.java 1476119 
  trunk/core/src/main/java/org/apache/oozie/event/MemoryEventQueue.java 1476119 
  trunk/core/src/main/java/org/apache/oozie/event/WorkflowActionEvent.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/event/WorkflowJobEvent.java 1476119 
  
trunk/core/src/main/java/org/apache/oozie/event/listener/JobEventListener.java 
1476119 
  
trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLACalculatorGetJPAExecutor.java
 PRE-CREATION 
  
trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLACalculatorInsertUpdateJPAExecutor.java
 PRE-CREATION 
  
trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLADashboardGetJPAExecutor.java
 PRE-CREATION 
  
trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLADashboardInsertUpdateJPAExecutor.java
 PRE-CREATION 
  
trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLAEventInsertJPAExecutor.java
 1476119 
  
trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLAEventsGetForFilterJPAExecutor.java
 1476119 
  
trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLAEventsGetForSeqIdJPAExecutor.java
 1476119 
  
trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLAEventsGetJPAExecutor.java
 1476119 
  trunk/core/src/main/java/org/apache/oozie/jms/JMSJobEventListener.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/service/EventHandlerService.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/service/JPAService.java 1476119 
  trunk/core/src/main/java/org/apache/oozie/servlet/SLAServlet.java 1476119 
  trunk/core/src/main/java/org/apache/oozie/sla/SLACalcObj.java PRE-CREATION 
  trunk/core/src/main/java/org/apache/oozie/sla/SLACalculationInfo.java 
PRE-CREATION 
  trunk/core/src/main/java/org/apache/oozie/sla/SLACalculatorBean.java 
PRE-CREATION 
  trunk/core/src/main/java/org/apache/oozie/sla/SLADashboardBean.java 
PRE-CREATION 
  trunk/core/src/main/java/org/apache/oozie/sla/SLAEventBean.java PRE-CREATION 
  trunk/core/src/main/java/org/apache/oozie/sla/listener/SLAEventListener.java 
PRE-CREATION 
  
trunk/core/src/main/java/org/apache/oozie/sla/listener/SLAStatusEventListener.java
 PRE-CREATION 
  trunk/core/src/main/java/org/apache/oozie/sla/service/SLAService.java 
PRE-CREATION 
  trunk/core/src/main/java/org/apache/oozie/store/SLAStore.java 1476119 
  trunk/core/src/main/java/org/apache/oozie/util/SLAOperations.java 
PRE-CREATION 
  trunk/core/src/main/java/org/apache/oozie/util/db/SLADbOperations.java 
1476119 
  trunk/core/src/main/java/org/apache/oozie/util/db/SLADbXOperations.java 
1476119 
  trunk/core/src/main/resources/META-INF/oozie-derby-orm.xml 1476119 
  trunk/core/src/main/resources/META-INF/oozie-hsqldb-orm.xml 1476119 
  trunk/core/src/main/resources/META-INF/oozie-mysql-orm.xml 1476119 
  trunk/core/src/main/resources/META-INF/oozie-postgresql-orm.xml 1476119 
  trunk/core/src/main/resources/META-INF/persistence.xml 1476119 
  trunk/core/src/main/resources/oozie-default.xml 1476119 
  
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java
 1476119 
  
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionExecutor.java
 1476119 
  
trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordActionMaterializeCommand.java
 1476119 
  
trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordMaterializeTransitionXCommand.java
 1476119 
  
trunk/core/src/test/java/org/apache/oozie/dependency/TestURIHandlerService.java 
1476119 
  trunk/core/src/test/java/org/apache/oozie/event/TestEventGeneration.java 
1476119 
  trunk/core/src/test/java/org/apache/oozie/event/TestEventQueue.java 1476119 
  
trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestSLACalculatorJPAExecutor.java
 PRE-CREATION 
  
trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestSLADashboardJPAExecutor.java
 PRE-CREATION 
  
trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestSLAEventsGetForFilterJPAExecutor.java
 1476119 
  
trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestSLAEventsGetForSeqIdJPAExecutor.java
 1476119 
  
trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestSLAEventsGetJPAExecutor.java
 1476119 
  trunk/core/src/test/java/org/apache/oozie/jms/TestJMSJobEventListener.java 
1476119 
  
trunk/core/src/test/java/org/apache/oozie/service/TestEventHandlerService.java 
1476119 
  trunk/core/src/test/java/org/apache/oozie/sla/TestSLAEventGeneration.java 
PRE-CREATION 
  trunk/core/src/test/java/org/apache/oozie/sla/TestSLAService.java 
PRE-CREATION 
  trunk/core/src/test/java/org/apache/oozie/sla/TestSLAStatusEventListener.java 
PRE-CREATION 
  trunk/core/src/test/java/org/apache/oozie/store/TestSLAStore.java 1476119 
  trunk/core/src/test/java/org/apache/oozie/test/XDataTestCase.java 1476119 
  trunk/core/src/test/java/org/apache/oozie/test/XTestCase.java 1476119 
  trunk/webapp/pom.xml 1476119 

Diff: https://reviews.apache.org/r/10569/diff/


Testing
-------

ongoing


Thanks,

Mona Chitnis

Reply via email to