----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5093/#review7811 -----------------------------------------------------------
Ship it! +1 - Mohammad On 2012-05-11 19:27:34, Alejandro Abdelnur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5093/ > ----------------------------------------------------------- > > (Updated 2012-05-11 19:27:34) > > > Review request for oozie. > > > Summary > ------- > > Instead using a shutdownhook, the patch uses a monitor thread that detects > that no testcase is running for X seconds and then triggers a shutdown of the > minicluster > > > This addresses bug OOZIE-835. > https://issues.apache.org/jira/browse/OOZIE-835 > > > Diffs > ----- > > trunk/core/src/test/java/org/apache/oozie/test/XTestCase.java 1337157 > > Diff: https://reviews.apache.org/r/5093/diff > > > Testing > ------- > > run all testcases with hadoop 1 and hadoop 2 and test JVM shutdown properly > > > Thanks, > > Alejandro > >
