> On July 24, 2013, 8:33 a.m., Mona Chitnis wrote: > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java, > > line 766 > > <https://reviews.apache.org/r/12866/diff/1/?file=325542#file325542line766> > > > > this block can be combined with the one above nested within the > > if(ended) block, since it only involves checking eventProc (not getting > > overwritten above there) and setting status - event status in addition to > > SLAStatus, and sending out the event
If set there, it will be overwritten by DURATION_MET or MISS status > On July 24, 2013, 8:33 a.m., Mona Chitnis wrote: > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java, > > line 779 > > <https://reviews.apache.org/r/12866/diff/1/?file=325542#file325542line779> > > > > This else block is not pertaining to Database operations. How exactly > > is it useful for HA, since the slaCalc object is still passed from > > in-memory reference of same server? For cases the job is started but not ended - Rohini ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12866/#review23743 ----------------------------------------------------------- On July 26, 2013, 9:52 p.m., Rohini Palaniswamy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12866/ > ----------------------------------------------------------- > > (Updated July 26, 2013, 9:52 p.m.) > > > Review request for oozie. > > > Bugs: OOZIE-1472 > https://issues.apache.org/jira/browse/OOZIE-1472 > > > Repository: oozie > > > Description > ------- > > Changes done: > - Confirming against DB for start and duration miss also. This prevents wrong > events getting generated when lesser threads are configured. Also will be > required in case of HA. > - Still see SLA tests failing sometime. But this has reduced the probability > a lot. Will look at completely addressing it in another JIRA. > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/oozie/trunk/client/src/main/java/org/apache/oozie/client/event/message/SLAMessage.java > 1507430 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/service/EventHandlerService.java > 1507430 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/sla/SLACalculator.java > 1507430 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java > 1507430 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/sla/SLAOperations.java > 1507430 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/sla/listener/SLAEmailEventListener.java > 1507430 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/resources/oozie-log4j.properties > 1507430 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/service/TestJMSTopicService.java > 1507430 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/service/TestPartitionDependencyManagerEhcache.java > 1507430 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/sla/TestSLACalculatorMemory.java > 1507430 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/sla/TestSLAEmailEventListener.java > 1507430 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/sla/TestSLAEventGeneration.java > 1507430 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/sla/TestSLAService.java > 1507430 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/resources/wf-job-sla.xml > 1507430 > > http://svn.apache.org/repos/asf/oozie/trunk/docs/src/site/twiki/DG_SLAMonitoring.twiki > 1507430 > > Diff: https://reviews.apache.org/r/12866/diff/ > > > Testing > ------- > > Unit tests pass > > > Thanks, > > Rohini Palaniswamy > >
