This is an automated email from the ASF dual-hosted git repository.

stoty pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/phoenix.git


The following commit(s) were added to refs/heads/master by this push:
     new 5071df2875 PHOENIX-6904 Clean up some dependencies
5071df2875 is described below

commit 5071df287537a44677a4dba21da74011873471cf
Author: Istvan Toth <st...@apache.org>
AuthorDate: Tue Mar 14 08:25:46 2023 +0100

    PHOENIX-6904 Clean up some dependencies
---
 phoenix-core/pom.xml                               | 35 +++++++---------------
 .../org/apache/phoenix/jdbc/FailoverPolicy.java    |  5 ++--
 pom.xml                                            | 30 ++++++++++++++++---
 3 files changed, 38 insertions(+), 32 deletions(-)

diff --git a/phoenix-core/pom.xml b/phoenix-core/pom.xml
index e6d7c35670..9e76f75967 100644
--- a/phoenix-core/pom.xml
+++ b/phoenix-core/pom.xml
@@ -276,18 +276,10 @@
       <groupId>org.apache.hadoop</groupId>
       <artifactId>hadoop-mapreduce-client-core</artifactId>
     </dependency>
-    <dependency>
-      <groupId>org.apache.hadoop</groupId>
-      <artifactId>hadoop-hdfs</artifactId>
-    </dependency>
     <dependency>
       <groupId>org.apache.hadoop</groupId>
       <artifactId>hadoop-hdfs-client</artifactId>
     </dependency>
-    <dependency>
-      <groupId>org.apache.hadoop</groupId>
-      <artifactId>hadoop-annotations</artifactId>
-    </dependency>
 
     <!-- Hadoop test dependencies -->
     <dependency>
@@ -298,14 +290,18 @@
     <dependency>
       <groupId>org.apache.hadoop</groupId>
       <artifactId>hadoop-hdfs</artifactId>
-      <version>${hadoop.version}</version>
       <scope>test</scope>
+    </dependency>
+    <dependency>
+      <groupId>org.apache.hadoop</groupId>
+      <artifactId>hadoop-hdfs</artifactId>
       <type>test-jar</type>
+      <classifier>tests</classifier>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.apache.hadoop</groupId>
       <artifactId>hadoop-common</artifactId>
-      <version>${hadoop.version}</version>
       <scope>test</scope>
       <type>test-jar</type>
     </dependency>
@@ -398,6 +394,10 @@
       <groupId>org.apache.zookeeper</groupId>
       <artifactId>zookeeper</artifactId>
     </dependency>
+    <dependency>
+        <groupId>org.apache.zookeeper</groupId>
+        <artifactId>zookeeper-jute</artifactId>
+    </dependency>
 
     <!-- Transaction dependencies -->
     <!-- Omid dependencies -->
@@ -553,10 +553,6 @@
       <groupId>com.lmax</groupId>
       <artifactId>disruptor</artifactId>
     </dependency>
-    <dependency>
-        <groupId>org.apache.zookeeper</groupId>
-        <artifactId>zookeeper-jute</artifactId>
-    </dependency>
     <dependency>
       <groupId>org.jruby.joni</groupId>
       <artifactId>joni</artifactId>
@@ -568,17 +564,6 @@
     <dependency>
       <groupId>org.hdrhistogram</groupId>
       <artifactId>HdrHistogram</artifactId>
-      <version>2.1.12</version>
-    </dependency>
-    <dependency>
-      <groupId>com.sun.xml.bind</groupId>
-      <artifactId>jaxb-impl</artifactId>
-      <version>2.2.3-1</version>
-    </dependency>
-    <dependency>
-      <groupId>org.apache.httpcomponents</groupId>
-      <artifactId>httpcore</artifactId>
-      <version>4.4.4</version>
     </dependency>
 
     <!-- Other test dependencies -->
diff --git 
a/phoenix-core/src/main/java/org/apache/phoenix/jdbc/FailoverPolicy.java 
b/phoenix-core/src/main/java/org/apache/phoenix/jdbc/FailoverPolicy.java
index 52e9da5159..4d81bfd9a2 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/jdbc/FailoverPolicy.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/jdbc/FailoverPolicy.java
@@ -21,10 +21,10 @@ import java.sql.Connection;
 import java.util.Properties;
 
 import org.apache.commons.lang3.StringUtils;
-import org.apache.hadoop.classification.InterfaceStability;
-import org.apache.http.annotation.Immutable;
 import org.apache.phoenix.exception.FailoverSQLException;
 
+import net.jcip.annotations.Immutable;
+
 /**
  * A failover policy defines how failover connection deals with existing 
connections in case of
  * cluster role transition is detected.
@@ -94,7 +94,6 @@ public interface FailoverPolicy {
      * In case of {@link FailoverSQLException} exception, clients can still 
re-connect to this HA
      * group by creating a new failover connection, OR call static method 
failover() explicitly.
      */
-    @InterfaceStability.Unstable
     class FailoverToActivePolicy implements FailoverPolicy {
         public static final String NAME = "active";
         private static final int MAX_FAILOVER_COUNT_DEFAULT = 3;
diff --git a/pom.xml b/pom.xml
index 90be0f4a14..96b9658802 100644
--- a/pom.xml
+++ b/pom.xml
@@ -131,6 +131,7 @@
     <mockito.version>1.10.19</mockito.version>
     <junit.version>4.13.1</junit.version>
     <hamcrest.version>1.3</hamcrest.version>
+    <hdrhistogram.version>2.1.12</hdrhistogram.version>
 
     <!-- Plugin versions -->
     <maven-eclipse-plugin.version>2.9</maven-eclipse-plugin.version>
@@ -386,10 +387,6 @@
                 <ignoredUnusedDeclaredDependency>
                   org.apache.logging.log4j:log4j-core
                 </ignoredUnusedDeclaredDependency>
-                <ignoredUnusedDeclaredDependency>
-                  <!-- Needed for Hbase 2.5 -->
-                  com.sun.xml.bind:jaxb-impl
-                </ignoredUnusedDeclaredDependency>
               </ignoredUnusedDeclaredDependencies>
               <ignoredUsedUndeclaredDependencies>
                 <ignoredUsedUndeclaredDependency>
@@ -734,6 +731,26 @@
           </exclusion>
         </exclusions>
       </dependency>
+      <dependency>
+        <groupId>org.apache.hadoop</groupId>
+        <artifactId>hadoop-common</artifactId>
+        <type>test-jar</type>
+        <version>${hadoop.version}</version>
+        <exclusions>
+          <exclusion>
+            <groupId>org.apache.htrace</groupId>
+            <artifactId>htrace-core4</artifactId>
+          </exclusion>
+          <exclusion>
+            <groupId>log4j</groupId>
+            <artifactId>log4j</artifactId>
+          </exclusion>
+          <exclusion>
+            <groupId>org.slf4j</groupId>
+            <artifactId>slf4j-log4j12</artifactId>
+          </exclusion>
+        </exclusions>
+      </dependency>
       <dependency>
         <groupId>org.apache.hadoop</groupId>
         <artifactId>hadoop-mapreduce-client-core</artifactId>
@@ -1433,6 +1450,11 @@
         <artifactId>jcodings</artifactId>
         <version>${jcodings.version}</version>
       </dependency>
+      <dependency>
+        <groupId>org.hdrhistogram</groupId>
+        <artifactId>HdrHistogram</artifactId>
+        <version>${hdrhistogram.version}</version>
+      </dependency>
 
       <!-- Other test dependencies -->
       <dependency>

Reply via email to