-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5093/
-----------------------------------------------------------

(Updated 2012-05-11 19:27:34.298659)


Review request for oozie.


Changes
-------

Per Mohammad's suggestion, adding a system property that can modify the 
shutdown wait time of the monitor, defaulting to 10 seconds.

Per (IM) Virag's suggestion, the setupEmbeddedHadoop() minicluster 
initialization is try/catch and a shutdown is issued if an initialization error 
occurs.


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 (updated)
-----

  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

Reply via email to