----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10569/#review19547 -----------------------------------------------------------
First pass at review comments. Few classes still left. Will finish them in the second pass. Please add a note in Description that the reason of merging SLADBOperations and SLADBxoperations to SLAOperations is to remove duplicate code. trunk/client/src/main/java/org/apache/oozie/client/event/Event.java <https://reviews.apache.org/r/10569/#comment41424> abstract Event with just msgType - rest to the extending classes trunk/client/src/main/java/org/apache/oozie/client/event/SLAEvent.java <https://reviews.apache.org/r/10569/#comment41425> getJobStatus trunk/client/src/main/java/org/apache/oozie/client/event/SLAEvent.java <https://reviews.apache.org/r/10569/#comment41426> getJobId trunk/client/src/main/java/org/apache/oozie/client/event/SLAEvent.java <https://reviews.apache.org/r/10569/#comment41429> alert email. trunk/client/src/main/java/org/apache/oozie/client/event/SLAEvent.java <https://reviews.apache.org/r/10569/#comment41431> Need nominalTime. But keep it optional. Make a note to add documentation that not required to be specified in case of co-ordinator. trunk/client/src/main/resources/oozie-sla-0.2.xsd <https://reviews.apache.org/r/10569/#comment41577> minoccurs 0 trunk/client/src/main/resources/oozie-sla-0.2.xsd <https://reviews.apache.org/r/10569/#comment41433> max-duration trunk/client/src/main/resources/oozie-sla-0.2.xsd <https://reviews.apache.org/r/10569/#comment41576> minoccurs 0 trunk/client/src/main/resources/oozie-sla-0.2.xsd <https://reviews.apache.org/r/10569/#comment41434> alert-email. minoccurs 0 trunk/client/src/main/resources/oozie-sla-0.2.xsd <https://reviews.apache.org/r/10569/#comment41432> indentation trunk/core/src/main/java/org/apache/oozie/client/rest/sla/JsonSLAEvent.java <https://reviews.apache.org/r/10569/#comment41435> SLA_REGISTRATION. Class name also JsonSLARegistrationEvent trunk/core/src/main/java/org/apache/oozie/client/rest/sla/JsonSLAEvent.java <https://reviews.apache.org/r/10569/#comment41438> Whitespace trunk/core/src/main/java/org/apache/oozie/command/coord/SLAEventsXCommand.java <https://reviews.apache.org/r/10569/#comment41443> Remnant from old patch? trunk/core/src/main/java/org/apache/oozie/command/wf/KillXCommand.java <https://reviews.apache.org/r/10569/#comment41463> Needs to be put back or the commented code below uncommented trunk/core/src/main/java/org/apache/oozie/command/wf/KillXCommand.java <https://reviews.apache.org/r/10569/#comment41462> Commented code to be removed trunk/core/src/main/java/org/apache/oozie/command/wf/SignalXCommand.java <https://reviews.apache.org/r/10569/#comment41464> Shouldn't be in submit instead of SignalX? trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLACalculatorGetJPAExecutor.java <https://reviews.apache.org/r/10569/#comment41552> Should we create a new package sla under executor/jpa? The package has too many classes. trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLACalculatorGetJPAExecutor.java <https://reviews.apache.org/r/10569/#comment41549> whitespace trunk/core/src/main/java/org/apache/oozie/service/EventHandlerService.java <https://reviews.apache.org/r/10569/#comment41557> (numWorkers + 5) trunk/core/src/main/java/org/apache/oozie/service/SchemaService.java <https://reviews.apache.org/r/10569/#comment41560> Accidental change needs to be reverted. trunk/core/src/main/java/org/apache/oozie/service/SchemaService.java <https://reviews.apache.org/r/10569/#comment41561> New sla xsd needs to be added trunk/core/src/main/java/org/apache/oozie/sla/SLACalcStatus.java <https://reviews.apache.org/r/10569/#comment41562> Not required trunk/core/src/main/java/org/apache/oozie/sla/SLAEventBean.java <https://reviews.apache.org/r/10569/#comment41571> Sequence id not required trunk/core/src/main/java/org/apache/oozie/sla/SLAEventBean.java <https://reviews.apache.org/r/10569/#comment41567> SLARegistrationBean trunk/core/src/main/java/org/apache/oozie/sla/SLAEventBean.java <https://reviews.apache.org/r/10569/#comment41572> Status here? trunk/core/src/main/java/org/apache/oozie/sla/SLASummaryBean.java <https://reviews.apache.org/r/10569/#comment41573> Can we have one more status column that summarizes the state based on end time - MET, MISS, RUNNING, NOT_STARTED - Rohini Palaniswamy On May 3, 2013, 10:09 a.m., Mona Chitnis wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10569/ > ----------------------------------------------------------- > > (Updated May 3, 2013, 10:09 a.m.) > > > Review request for oozie. > > > 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 > ----- > > trunk/client/src/main/java/org/apache/oozie/AppType.java PRE-CREATION > trunk/client/src/main/java/org/apache/oozie/client/SLAEvent.java 1478670 > trunk/client/src/main/java/org/apache/oozie/client/event/Event.java 1478670 > trunk/client/src/main/java/org/apache/oozie/client/event/JobEvent.java > 1478670 > trunk/client/src/main/java/org/apache/oozie/client/event/SLAEvent.java > 1478670 > > trunk/client/src/main/java/org/apache/oozie/client/event/jms/JSONMessageDeserializer.java > 1478670 > > trunk/client/src/main/java/org/apache/oozie/client/event/jms/MessageDeserializer.java > 1478670 > > trunk/client/src/main/java/org/apache/oozie/client/event/message/CoordinatorActionMessage.java > 1478670 > > trunk/client/src/main/java/org/apache/oozie/client/event/message/EventMessage.java > 1478670 > > trunk/client/src/main/java/org/apache/oozie/client/event/message/JobMessage.java > 1478670 > > trunk/client/src/main/java/org/apache/oozie/client/event/message/WorkflowJobMessage.java > 1478670 > trunk/client/src/main/resources/oozie-sla-0.2.xsd PRE-CREATION > trunk/core/pom.xml 1478670 > trunk/core/src/main/java/org/apache/oozie/ErrorCode.java 1478670 > trunk/core/src/main/java/org/apache/oozie/SLAEventBean.java 1478670 > trunk/core/src/main/java/org/apache/oozie/client/rest/JsonSLAEvent.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/client/rest/sla/JsonSLAEvent.java > PRE-CREATION > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionCheckXCommand.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionInputCheckXCommand.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionMaterializeCommand.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionStartXCommand.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionTimeOutXCommand.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionUpdateXCommand.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordMaterializeTransitionXCommand.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordPushDependencyCheckXCommand.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordRerunXCommand.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordSuspendXCommand.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordinatorXCommand.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/command/coord/SLAEventsXCommand.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/command/wf/ActionEndXCommand.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/command/wf/ActionKillXCommand.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/command/wf/ActionStartXCommand.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/command/wf/KillXCommand.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/command/wf/ResumeXCommand.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/command/wf/SignalXCommand.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/command/wf/SubmitXCommand.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/command/wf/SuspendXCommand.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/command/wf/WorkflowXCommand.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/event/BundleJobEvent.java 1478670 > trunk/core/src/main/java/org/apache/oozie/event/CoordinatorActionEvent.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/event/CoordinatorJobEvent.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/event/EventQueue.java 1478670 > trunk/core/src/main/java/org/apache/oozie/event/MemoryEventQueue.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/event/WorkflowActionEvent.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/event/WorkflowJobEvent.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/event/listener/JobEventListener.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLACalculationInsertUpdateJPAExecutor.java > PRE-CREATION > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLACalculatorGetJPAExecutor.java > PRE-CREATION > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLAEventInsertJPAExecutor.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLAEventsGetForFilterJPAExecutor.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLAEventsGetForSeqIdJPAExecutor.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLAEventsGetJPAExecutor.java > 1478670 > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/SLASummaryGetJPAExecutor.java > PRE-CREATION > trunk/core/src/main/java/org/apache/oozie/jms/JMSJobEventListener.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/service/EventHandlerService.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/service/JPAService.java 1478670 > trunk/core/src/main/java/org/apache/oozie/service/SchedulerService.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/service/SchemaService.java > 1478670 > trunk/core/src/main/java/org/apache/oozie/servlet/SLAServlet.java 1478670 > trunk/core/src/main/java/org/apache/oozie/sla/SLACalcStatus.java > PRE-CREATION > trunk/core/src/main/java/org/apache/oozie/sla/SLACalculator.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/SLACalculatorMem.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/SLASummaryBean.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/SLAJobEventListener.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 1478670 > 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 > 1478670 > trunk/core/src/main/java/org/apache/oozie/util/db/SLADbXOperations.java > 1478670 > trunk/core/src/main/resources/META-INF/oozie-derby-orm.xml 1478670 > trunk/core/src/main/resources/META-INF/oozie-hsqldb-orm.xml 1478670 > trunk/core/src/main/resources/META-INF/oozie-mysql-orm.xml 1478670 > trunk/core/src/main/resources/META-INF/oozie-postgresql-orm.xml 1478670 > trunk/core/src/main/resources/META-INF/persistence.xml 1478670 > trunk/core/src/main/resources/oozie-default.xml 1478670 > > trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordActionMaterializeCommand.java > 1478670 > > trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordMaterializeTransitionXCommand.java > 1478670 > trunk/core/src/test/java/org/apache/oozie/event/TestEventGeneration.java > 1478670 > trunk/core/src/test/java/org/apache/oozie/event/TestEventQueue.java 1478670 > > trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestSLACalculationJPAExecutor.java > PRE-CREATION > trunk/core/src/test/java/org/apache/oozie/jms/TestJMSJobEventListener.java > 1478670 > > trunk/core/src/test/java/org/apache/oozie/service/TestEventHandlerService.java > 1478670 > 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/test/XTestCase.java 1478670 > > Diff: https://reviews.apache.org/r/10569/diff/ > > > Testing > ------- > > ongoing > > > Thanks, > > Mona Chitnis > >
