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/hbase-connectors.git


The following commit(s) were added to refs/heads/master by this push:
     new b850f82  HBASE-29286 Use thirdparty protoc version when generating 
spark connector protobuf classes (#143)
b850f82 is described below

commit b850f821409c0b27b71a3dc33e47bcccbd009e8e
Author: Istvan Toth <[email protected]>
AuthorDate: Wed May 7 10:57:37 2025 +0200

    HBASE-29286 Use thirdparty protoc version when generating spark connector 
protobuf classes (#143)
---
 pom.xml                            | 9 ++-------
 spark/hbase-spark-protocol/pom.xml | 1 +
 spark/pom.xml                      | 2 +-
 3 files changed, 4 insertions(+), 8 deletions(-)

diff --git a/pom.xml b/pom.xml
index 81c3586..dd9ce97 100644
--- a/pom.xml
+++ b/pom.xml
@@ -142,6 +142,8 @@
     <junit.version>4.13.2</junit.version>
     <mockito-all.version>1.8.5</mockito-all.version>
     <hbase-thirdparty.version>4.1.4</hbase-thirdparty.version>
+    <!-- Must match the version in hbase-thirdparty above ! -->
+    <thirdparty.protobuf.version>3.21.12</thirdparty.protobuf.version>
     <hadoop-two.version>2.10.0</hadoop-two.version>
     <hadoop-three.version>3.2.4</hadoop-three.version>
     <hadoop.version>${hadoop-three.version}</hadoop.version>
@@ -156,8 +158,6 @@
     <enforcer.version>3.0.0</enforcer.version>
     <extra.enforcer.version>1.5.1</extra.enforcer.version>
     
<restrict-imports.enforcer.version>0.14.0</restrict-imports.enforcer.version>
-    <!--Internally we use a different version of protobuf. See 
hbase-protocol-shaded-->
-    <external.protobuf.version>2.5.0</external.protobuf.version>
     <protobuf.plugin.version>0.5.0</protobuf.plugin.version>
     <commons-io.version>2.18.0</commons-io.version>
     <avro.version>1.11.4</avro.version>
@@ -179,11 +179,6 @@
   </properties>
   <dependencyManagement>
     <dependencies>
-      <dependency>
-        <groupId>com.google.protobuf</groupId>
-        <artifactId>protobuf-java</artifactId>
-        <version>${external.protobuf.version}</version>
-      </dependency>
       <dependency>
         <groupId>org.apache.commons</groupId>
         <artifactId>commons-lang3</artifactId>
diff --git a/spark/hbase-spark-protocol/pom.xml 
b/spark/hbase-spark-protocol/pom.xml
index 20e752c..da94692 100644
--- a/spark/hbase-spark-protocol/pom.xml
+++ b/spark/hbase-spark-protocol/pom.xml
@@ -36,6 +36,7 @@
     <dependency>
       <groupId>com.google.protobuf</groupId>
       <artifactId>protobuf-java</artifactId>
+      <version>${thirdparty.protobuf.version}</version>
     </dependency>
   </dependencies>
 
diff --git a/spark/pom.xml b/spark/pom.xml
index 3f1eb21..b770e6b 100644
--- a/spark/pom.xml
+++ b/spark/pom.xml
@@ -84,7 +84,7 @@
           <artifactId>protobuf-maven-plugin</artifactId>
           <version>${protobuf.plugin.version}</version>
           <configuration>
-            
<protocArtifact>com.google.protobuf:protoc:${external.protobuf.version}:exe:${os.detected.classifier}</protocArtifact>
+            
<protocArtifact>com.google.protobuf:protoc:${thirdparty.protobuf.version}:exe:${os.detected.classifier}</protocArtifact>
             <protoSourceRoot>${basedir}/src/main/protobuf/</protoSourceRoot>
             <clearOutputDirectory>false</clearOutputDirectory>
             <checkStaleness>true</checkStaleness>

Reply via email to