> On July 24, 2013, 8:48 a.m., Mona Chitnis wrote: > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/service/TestPartitionDependencyManagerEhcache.java, > > line 80 > > <https://reviews.apache.org/r/12866/diff/1/?file=325547#file325547line80> > > > > Is 100 seconds a reasonable enough buffer to add for this flaky test?
Yes. - Rohini ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12866/#review23744 ----------------------------------------------------------- On July 26, 2013, 11:31 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, 11:31 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 > >
