[Java] Downgrade protobuf to 3.4.0

The recent bump in the Protobuf version breaks el6
compatibility due to a bug in the Maven delpoyed
protoc artifacts.

This patch downgrades to the latest compatible version
and adds the issue url for tracking.

https://github.com/google/protobuf/issues/4109

Change-Id: I1c23138bb030a3d5069dd8efa9e7231c3f2fd339
Reviewed-on: http://gerrit.cloudera.org:8080/9304
Reviewed-by: Todd Lipcon <t...@apache.org>
Tested-by: Kudu Jenkins


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/6f30a311
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/6f30a311
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/6f30a311

Branch: refs/heads/master
Commit: 6f30a3118e351bcb11a8ca2aa3f0e2ee8d882cc6
Parents: f9b2ccc
Author: Grant Henke <granthe...@gmail.com>
Authored: Tue Feb 13 14:20:39 2018 -0600
Committer: Grant Henke <granthe...@gmail.com>
Committed: Tue Feb 13 21:04:11 2018 +0000

----------------------------------------------------------------------
 java/gradle/dependencies.gradle | 4 +++-
 java/pom.xml                    | 4 +++-
 2 files changed, 6 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/6f30a311/java/gradle/dependencies.gradle
----------------------------------------------------------------------
diff --git a/java/gradle/dependencies.gradle b/java/gradle/dependencies.gradle
index 1dd2afe..12635b5 100755
--- a/java/gradle/dependencies.gradle
+++ b/java/gradle/dependencies.gradle
@@ -46,7 +46,9 @@ versions += [
     netty          : "3.10.6.Final",
     parquet        : "1.9.0",
     pmd            : "5.8.1",
-    protobuf       : "3.5.1",
+    // Upgrading protobuf breaks el6 compatibility
+    // https://github.com/google/protobuf/issues/4109
+    protobuf       : "3.4.0",
     scala          : "2.11.12",
     scalatest      : "3.0.5",
     slf4j          : "1.7.25",

http://git-wip-us.apache.org/repos/asf/kudu/blob/6f30a311/java/pom.xml
----------------------------------------------------------------------
diff --git a/java/pom.xml b/java/pom.xml
index c798530..3355201 100644
--- a/java/pom.xml
+++ b/java/pom.xml
@@ -83,7 +83,9 @@
         <murmur.version>1.0.0</murmur.version>
         <netty.version>3.10.6.Final</netty.version>
         <parquet.version>1.9.0</parquet.version>
-        <protobuf.version>3.5.1</protobuf.version>
+        <!-- Upgrading protobuf breaks el6 compatibility
+             https://github.com/google/protobuf/issues/4109 -->
+        <protobuf.version>3.4.0</protobuf.version>
         <slf4j.version>1.7.25</slf4j.version>
         <sparkavro.version>3.2.0</sparkavro.version>
         <yetus.version>0.7.0</yetus.version>

Reply via email to