RANGER-4418: remove unnecessary dependencies

commons-text already has commons-lang3
apache · Oct 20, 2024 · 44027ac · 44027ac
1 parent 37bed03
commit 44027ac
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 50 deletions.
diff --git a/agents-audit/pom.xml b/agents-audit/pom.xml
@@ -42,11 +42,6 @@
             <artifactId>commons-lang</artifactId>
             <version>${commons.lang.version}</version>
         </dependency>
-        <dependency>
-            <groupId>org.apache.commons</groupId>
-            <artifactId>commons-lang3</artifactId>
-            <version>${commons.lang3.version}</version>
-        </dependency>
         <dependency>
             <groupId>commons-collections</groupId>
             <artifactId>commons-collections</artifactId>
@@ -62,10 +57,6 @@
             <artifactId>hadoop-client-runtime</artifactId>
             <version>${hadoop.version}</version>
         </dependency>
-        <dependency>
-            <groupId>com.google.code.gson</groupId>
-            <artifactId>gson</artifactId>
-        </dependency>
         <dependency>
             <groupId>org.apache.commons</groupId>
             <artifactId>commons-text</artifactId>

diff --git a/agents-common/pom.xml b/agents-common/pom.xml
@@ -65,9 +65,13 @@
             <version>${javax.servlet.version}</version>
         </dependency>
         <dependency>
-            <groupId>commons-io</groupId>
-            <artifactId>commons-io</artifactId>
-            <version>${commons.io.version}</version>
+            <groupId>com.google.code.gson</groupId>
+            <artifactId>gson</artifactId>
+        </dependency>
+        <dependency>
+            <groupId>com.sun.xml.bind</groupId>
+            <artifactId>jaxb-impl</artifactId>
+            <version>${jaxb-impl.version}</version>
         </dependency>
         <dependency>
             <groupId>org.apache.hadoop</groupId>

diff --git a/agents-common/src/main/java/org/apache/ranger/plugin/util/RangerCache.java b/agents-common/src/main/java/org/apache/ranger/plugin/util/RangerCache.java
@@ -17,7 +17,7 @@
 
 package org.apache.ranger.plugin.util;
 
-import javax.annotation.Nonnull;
+import com.sun.istack.NotNull;
 import org.apache.ranger.plugin.util.AutoClosableLock.AutoClosableTryLock;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -363,7 +363,7 @@ private ThreadFactory createThreadFactory() {
             private final AtomicInteger number     = new AtomicInteger(1);
 
             @Override
-            public Thread newThread(@Nonnull Runnable r) {
+            public Thread newThread(@NotNull Runnable r) {
                 Thread t = new Thread(r, namePrefix + number.getAndIncrement());
 
                 if (!t.isDaemon()) {

diff --git a/credentialbuilder/pom.xml b/credentialbuilder/pom.xml
@@ -38,12 +38,6 @@
             <artifactId>commons-collections</artifactId>
             <version>${commons.collections.version}</version>
         </dependency>
-        <dependency>
-            <groupId>commons-io</groupId>
-            <artifactId>commons-io</artifactId>
-            <version>${commons.io.version}</version>
-            <scope>test</scope>
-        </dependency>
         <dependency>
             <groupId>org.apache.commons</groupId>
             <artifactId>commons-configuration2</artifactId>

diff --git a/hbase-agent/pom.xml b/hbase-agent/pom.xml
@@ -169,21 +169,6 @@
     		<artifactId>httpcore</artifactId>
     		<version>${httpcomponents.httpcore.version}</version>
 		</dependency>
-        <dependency>
-            <groupId>org.apache.commons</groupId>
-            <artifactId>commons-text</artifactId>
-            <version>${commons.text.version}</version>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.commons</groupId>
-            <artifactId>commons-configuration2</artifactId>
-            <version>${commons.configuration.version}</version>
-        </dependency>
-        <dependency>
-            <groupId>org.glassfish</groupId>
-            <artifactId>javax.el</artifactId>
-            <version>${javax.el.version}</version>
-        </dependency>
         <dependency>
             <groupId>io.opentelemetry</groupId>
             <artifactId>opentelemetry-api</artifactId>
@@ -338,21 +323,6 @@
             <version>${hadoop.version}</version>
             <scope>test</scope>
         </dependency>
-        <dependency>
-            <groupId>org.apache.commons</groupId>
-            <artifactId>commons-text</artifactId>
-            <version>${commons.text.version}</version>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.commons</groupId>
-            <artifactId>commons-configuration2</artifactId>
-            <version>${commons.configuration.version}</version>
-        </dependency>
-        <dependency>
-            <groupId>org.glassfish</groupId>
-            <artifactId>javax.el</artifactId>
-            <version>${javax.el.version}</version>
-        </dependency>
         <dependency>
             <groupId>org.slf4j</groupId>
             <artifactId>log4j-over-slf4j</artifactId>