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

(Updated Dec. 13, 2016, 9:57 p.m.)


Review request for oozie and AndrĂ¡s Piros.


Changes
-------

Address comments


Repository: oozie-git


Description
-------

Change the OozieShareLibCLI to perform a final 'rename' to avoid a condition 
where 'half' of the sharelib could exist during sharelib installation, if Oozie 
attempts to run a workflow at the same time as the sharelib installation. Oozie 
will only start if the sharelib exists, but this by itself is not enough since 
the sharelib could be in an unusable state.


Diffs (updated)
-----

  tools/src/main/java/org/apache/oozie/tools/OozieSharelibCLI.java 9aa4cb6 

Diff: https://reviews.apache.org/r/54557/diff/


Testing
-------

$ mvn test -Dtest=TestOozieSharelibCLI

Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 21.497 sec

Results :

Tests run: 4, Failures: 0, Errors: 0, Skipped: 0


Thanks,

Yishan Yang

Reply via email to