----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73281/#review223524 -----------------------------------------------------------
Ship it! Build succeeded with patch on java 8 and java 11. - Dhaval Shah On Sept. 21, 2021, 7:18 a.m., Martin Grigorov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73281/ > ----------------------------------------------------------- > > (Updated Sept. 21, 2021, 7:18 a.m.) > > > Review request for ranger. > > > Repository: ranger > > > Description > ------- > > Fix build due to hive exec and JDK 8 specific class > > > Diffs > ----- > > kms/src/main/java/org/apache/hadoop/crypto/key/RangerSafenetKeySecure.java > 371e367c1 > pom.xml 36036a6ad > 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/4/ > > > Testing > ------- > > 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 > >