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 489dce9ca0 PHOENIX-7356 Centralize and update versions for 
exclude-only dependencies
489dce9ca0 is described below

commit 489dce9ca00b7e275b8ae56d21bba2db896938be
Author: Istvan Toth <[email protected]>
AuthorDate: Thu Jul 11 09:55:57 2024 +0200

    PHOENIX-7356 Centralize and update versions for exclude-only dependencies
---
 phoenix-mapreduce-byo-shaded-hbase/pom.xml | 12 ++++--------
 phoenix-server/pom.xml                     | 12 ++++--------
 pom.xml                                    |  7 +++++++
 3 files changed, 15 insertions(+), 16 deletions(-)

diff --git a/phoenix-mapreduce-byo-shaded-hbase/pom.xml 
b/phoenix-mapreduce-byo-shaded-hbase/pom.xml
index 25a5c59d40..6dfaf8d31a 100644
--- a/phoenix-mapreduce-byo-shaded-hbase/pom.xml
+++ b/phoenix-mapreduce-byo-shaded-hbase/pom.xml
@@ -556,30 +556,26 @@
     <dependency>
       <groupId>org.apache.hbase.thirdparty</groupId>
       <artifactId>hbase-shaded-netty</artifactId>
-      <!-- random version, for exclusion only -->
-      <version>4.0.0</version>
+      <version>${hbase-thirdparty.excludeonly.version}</version>
       <scope>provided</scope>
     </dependency>
     <dependency>
       <groupId>org.apache.hbase.thirdparty</groupId>
       <artifactId>hbase-shaded-miscellaneous</artifactId>
-      <!-- random version, for exclusion only -->
-      <version>4.0.0</version>
+      <version>${hbase-thirdparty.excludeonly.version}</version>
       <scope>provided</scope>
     </dependency>
     <dependency>
       <groupId>org.apache.hbase.thirdparty</groupId>
       <artifactId>hbase-shaded-protobuf</artifactId>
-      <!-- random version, for exclusion only -->
-      <version>4.0.0</version>
+      <version>${hbase-thirdparty.excludeonly.version}</version>
       <scope>provided</scope>
     </dependency>
     <dependency>
        <!-- provided by Hadoop -->
       <groupId>org.apache.commons</groupId>
       <artifactId>commons-configuration2</artifactId>
-      <!-- random version, for exclusion only -->
-      <version>2.1.1</version>
+      <version>${commons-configuration2.excludeonly.version}</version>
       <scope>provided</scope>
     </dependency>
 
diff --git a/phoenix-server/pom.xml b/phoenix-server/pom.xml
index 01de657d8a..8c10c74c28 100644
--- a/phoenix-server/pom.xml
+++ b/phoenix-server/pom.xml
@@ -523,30 +523,26 @@
     <dependency>
       <groupId>org.apache.hbase.thirdparty</groupId>
       <artifactId>hbase-shaded-netty</artifactId>
-      <!-- random version, for exclusion only -->
-      <version>4.0.0</version>
+      <version>${hbase-thirdparty.excludeonly.version}</version>
       <scope>provided</scope>
     </dependency>
     <dependency>
       <groupId>org.apache.hbase.thirdparty</groupId>
       <artifactId>hbase-shaded-miscellaneous</artifactId>
-      <!-- random version, for exclusion only -->
-      <version>4.0.0</version>
+      <version>${hbase-thirdparty.excludeonly.version}</version>
       <scope>provided</scope>
     </dependency>
     <dependency>
       <groupId>org.apache.hbase.thirdparty</groupId>
       <artifactId>hbase-shaded-protobuf</artifactId>
-      <!-- random version, for exclusion only -->
-      <version>4.0.0</version>
+      <version>${hbase-thirdparty.excludeonly.version}</version>
       <scope>provided</scope>
     </dependency>
     <dependency>
       <!-- provided by Hadoop -->
       <groupId>org.apache.commons</groupId>
       <artifactId>commons-configuration2</artifactId>
-      <!-- random version, for exclusion only -->
-      <version>2.1.1</version>
+      <version>${commons-configuration2.excludeonly.version}</version>
       <scope>provided</scope>
     </dependency>
 
diff --git a/pom.xml b/pom.xml
index e3d5b9ac9b..5518da0760 100644
--- a/pom.xml
+++ b/pom.xml
@@ -142,6 +142,13 @@
     <junit.version>4.13.1</junit.version>
     <hdrhistogram.version>2.1.12</hdrhistogram.version>
 
+    <!-- These are only used for exclusion when shading, and the exact version 
is completely
+    irrelevant, but we need to keep them up to date to appease static 
checkers. While Phoenix does
+    use theses packages, it does not explicitly depend on them, but instead 
takes the transitve
+    dependencies from the Hadoop/HBase version used for building. -->
+    
<comons-configuration2.excludeonly.version>2.10.1</comons-configuration2.excludeonly.version>
+    
<hbase-thirdparty.excludeonly.version>4.1.7</hbase-thirdparty.excludeonly.version>
+
     <!-- Plugin versions -->
     <maven-eclipse-plugin.version>2.10</maven-eclipse-plugin.version>
     
<maven-build-helper-plugin.version>3.5.0</maven-build-helper-plugin.version>

Reply via email to