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

(Updated May 16, 2018, 3:37 p.m.)


Review request for oozie, András Piros and Denes Bodo.


Changes
-------

Created separate ShareLibExcluder class, filtering is done on the full path 
(excluding the sharelib root), modified tests according to behaviour change 
(these could be put into a separate class + JavaActionExecutor test rework may 
be needed). I tried to take into account your proposals, but I kindly ask you 
to review this new patch and waiting for your opinion about it. Thanks!


Bugs: OOZIE-1624
    https://issues.apache.org/jira/browse/OOZIE-1624


Repository: oozie-git


Description
-------

OOZIE-1624 Exclusion pattern for sharelib.


Diffs (updated)
-----

  core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java 
0ba3cbf9 
  core/src/main/java/org/apache/oozie/action/hadoop/ShareLibExcluder.java 
PRE-CREATION 
  core/src/main/java/org/apache/oozie/service/ShareLibService.java a901567d 
  core/src/test/java/org/apache/oozie/service/TestShareLibService.java d2441661 


Diff: https://reviews.apache.org/r/66656/diff/2/

Changes: https://reviews.apache.org/r/66656/diff/1-2/


Testing
-------

Tested on a cluster


Thanks,

Mate Juhasz

Reply via email to