-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63987/
-----------------------------------------------------------
(Updated Nov. 28, 2017, 10:41 a.m.)
Review request for ranger.
Changes
-------
Remove the mixing of the libraries in 'cred' and 'ews' folder for the ranger
admin
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 fd988116d
Diff: https://reviews.apache.org/r/63987/diff/3/
Changes: https://reviews.apache.org/r/63987/diff/2-3/
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
