----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5093/#review7810 -----------------------------------------------------------
trunk/core/src/test/java/org/apache/oozie/test/XTestCase.java <https://reviews.apache.org/r/5093/#comment17145> can we create a variable for 10 second ? use variable in place of hard-coded number here. - Mohammad On 2012-05-11 13:43:19, Alejandro Abdelnur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5093/ > ----------------------------------------------------------- > > (Updated 2012-05-11 13:43:19) > > > 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 > >
