----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73281/ -----------------------------------------------------------
(Updated April 16, 2021, 8:08 a.m.) Review request for ranger. Repository: ranger Description ------- Fix build due to hive exec and JDK 8 specific class Diffs (updated) ----- agents-audit/pom.xml 06ed81f55 kms/src/main/java/org/apache/hadoop/crypto/key/RangerSafenetKeySecure.java 371e367c1 pom.xml fe7812bd7 ugsync-util/pom.xml aa992cabf ugsync/pom.xml 07e0c5872 ugsync/src/test/java/org/apache/ranger/unixusersync/process/TestUnixUserGroupBuilder.java 30820ce33 Diff: https://reviews.apache.org/r/73281/diff/2/ Changes: https://reviews.apache.org/r/73281/diff/1-2/ Testing (updated) ------- Issue 1) org.apache.ranger.authorization.presto.authorizer.RangerSystemAccessControlFactory was not able to import com.google.common.base.Throwables.throwIfUnchecked because it was coming from an old copy of Guava's Throwables shaded in hive-exec. By replacing hive-exec with orc-core in agents-audit module all depending modules use their preferred version of Guava (26.0-jre) Issue 2) RangerSafenetKeySecure.java uses directly sun.security.pkcs11.SunPKCS11 - a class that is not available in Java 9+ and this breaks the compilation. This should use reflection to load SunPKCS11 class dynamically. Issue 3) JAXB is no more part of JDK 11+ Issue 4) JUnit 4 Assert.assertThat() is deprecated in favour of Hamcrest's MatcherAssert.assertThat(). This fixes a compilation issue with JDK11. Thanks, Martin Grigorov
