-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63987/
-----------------------------------------------------------
(Updated Dec. 7, 2017, 8:54 p.m.)
Review request for ranger.
Changes
-------
rebased patch
Bugs: RANGER-1905
https://issues.apache.org/jira/browse/RANGER-1905
Repository: ranger
Description
-------
As ranger-plugins-audit doesn't exclude guava from hadoop common, an older guava
is leaked into embeddedwebserver classpath, which somehow filtered out by the
assembly plugin.
If we explicitly exclude the old guava, and include the new (17.0) guava in
ranger-plugins-audit module, the generated tar.gz/zip distributions will
contain the necessary guava-17.0.jar in ews/lib path.
Diffs (updated)
-----
agents-audit/pom.xml c8bd1d8f3
embeddedwebserver/scripts/ranger-admin-services.sh 278aab6c9
src/main/assembly/plugin-atlas.xml 2c581f8e8
Diff: https://reviews.apache.org/r/63987/diff/4/
Changes: https://reviews.apache.org/r/63987/diff/3-4/
Testing
-------
Tested with executing:
mvn -DskipTests package assembly:assembly;
unzip -Z1 target/ranger-1.0.0-SNAPSHOT-admin.zip | sort > FILES.txt
And comparing the results with different versions. With the patch, the guava
jar appears correctly.
Thanks,
Zsombor Gegesy
