[kudu-CR] [java] Upgrade dependencies
Attila Bukor has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18159 ) Change subject: [java] Upgrade dependencies .. [java] Upgrade dependencies Updgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito - 3.8.0 -> 4.2.0 Minor version upgrades: - commons - 2.8.0 -> 2.11.0 - jmh - 1.28 -> 1.34 - micrometer - 1.6.5 -> 1.8.2 - protobuf - 3.15.6 -> 3.19.3 Maintenance version upgrades: - clojure - 1.10.1 -> 1.10.3 - clojure tools cli - 1.0.194 -> 1.0.206 - hadoop - 3.3.0 -> 3.3.1 - jetty - 9.4.38.v20210224 -> 9.4.44.v20210927 - netty - 4.1.65.Final -> 4.1.73.Final - scala -> 2.12.13 -> 2.12.15 - scalatest - 3.2.6 -> 3.2.10 - slf4j - 1.7.30 -> 1.7.33 - spark -> 3.1.1 -> 3.1.2 - spark2 - 2.4.7 -> 2.4.8 Gradle upgrades: - gradle - 6.8.3 -> 6.9.2 - gradle-versions-plugin - 0.38.0 -> 0.41.0 - osdetector-gradle-plugin - 1.6.2 -> 1.7.0 - protobuf-gradle-plugin - 0.8.15 -> 0.8.18 - nebula-clojure-plugin - 9.4.3 -> 10.1.1 - propdeps-plugin - 0.0.9.RELEASE -> 0.0.10.RELEASE - gradle-errorprone-plugin - 1.3.0 -> 2.0.2 - gradle-animalsniffer-plugin - 1.5.3 -> 1.5.4 - gson - 2.8.6 -> 2.8.9 - guava - 30.1.-jre -> 31.0.1-jre Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Reviewed-on: http://gerrit.cloudera.org:8080/18159 Tested-by: Attila Bukor Reviewed-by: Alexey Serbin --- M build-support/verify_jars.pl M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradlew M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduRestore.scala M java/kudu-client/src/main/java/org/apache/kudu/client/RpcProxy.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduClient.java M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/DefaultSource.scala M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/package.scala 10 files changed, 42 insertions(+), 41 deletions(-) Approvals: Attila Bukor: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 12 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan
[kudu-CR] [java] Upgrade dependencies
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18159 ) Change subject: [java] Upgrade dependencies .. Patch Set 11: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/18159/6/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala File java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala: http://gerrit.cloudera.org:8080/#/c/18159/6/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala@a157 PS6, Line 157: > Thanks for bringing this up, I thought our pre-commits tested on Spark2 as Cool, thank you very much for clarifying! -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 11 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan Gerrit-Comment-Date: Wed, 26 Jan 2022 06:34:49 + Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/18159 ) Change subject: [java] Upgrade dependencies .. Patch Set 11: Verified+1 > Patch Set 11: Verified-1 > > Build Failed > > http://jenkins.kudu.apache.org/job/kudu-gerrit/25003/ : FAILURE Unrelated failure -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 11 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan Gerrit-Comment-Date: Tue, 25 Jan 2022 17:24:48 + Gerrit-HasComments: No
[kudu-CR] [java] Upgrade dependencies
Attila Bukor has removed a vote on this change. Change subject: [java] Upgrade dependencies .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: deleteVote Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 11 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan
[kudu-CR] [java] Upgrade dependencies
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/18159 ) Change subject: [java] Upgrade dependencies .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/18159/6/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala File java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala: http://gerrit.cloudera.org:8080/#/c/18159/6/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala@a157 PS6, Line 157: > BTW, does the newly updated code still works fine with Spark 2 (if it have Thanks for bringing this up, I thought our pre-commits tested on Spark2 as well. It turned out that Scala 2.11 can't use Java concurrent ForkJoinPool, which is deprecated in 2.12, but it works without explicitly calling shutdown. -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 11 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan Gerrit-Comment-Date: Tue, 25 Jan 2022 16:50:19 + Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Hello Zoltan Chovan, Alexey Serbin, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/18159 to look at the new patch set (#11). Change subject: [java] Upgrade dependencies .. [java] Upgrade dependencies Updgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito - 3.8.0 -> 4.2.0 Minor version upgrades: - commons - 2.8.0 -> 2.11.0 - jmh - 1.28 -> 1.34 - micrometer - 1.6.5 -> 1.8.2 - protobuf - 3.15.6 -> 3.19.3 Maintenance version upgrades: - clojure - 1.10.1 -> 1.10.3 - clojure tools cli - 1.0.194 -> 1.0.206 - hadoop - 3.3.0 -> 3.3.1 - jetty - 9.4.38.v20210224 -> 9.4.44.v20210927 - netty - 4.1.65.Final -> 4.1.73.Final - scala -> 2.12.13 -> 2.12.15 - scalatest - 3.2.6 -> 3.2.10 - slf4j - 1.7.30 -> 1.7.33 - spark -> 3.1.1 -> 3.1.2 - spark2 - 2.4.7 -> 2.4.8 Gradle upgrades: - gradle - 6.8.3 -> 6.9.2 - gradle-versions-plugin - 0.38.0 -> 0.41.0 - osdetector-gradle-plugin - 1.6.2 -> 1.7.0 - protobuf-gradle-plugin - 0.8.15 -> 0.8.18 - nebula-clojure-plugin - 9.4.3 -> 10.1.1 - propdeps-plugin - 0.0.9.RELEASE -> 0.0.10.RELEASE - gradle-errorprone-plugin - 1.3.0 -> 2.0.2 - gradle-animalsniffer-plugin - 1.5.3 -> 1.5.4 - gson - 2.8.6 -> 2.8.9 - guava - 30.1.-jre -> 31.0.1-jre Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 --- M build-support/verify_jars.pl M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradlew M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduRestore.scala M java/kudu-client/src/main/java/org/apache/kudu/client/RpcProxy.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduClient.java M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/DefaultSource.scala M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/package.scala 10 files changed, 42 insertions(+), 41 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/59/18159/11 -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 11 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan
[kudu-CR] [java] Upgrade dependencies
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18159 ) Change subject: [java] Upgrade dependencies .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/18159/6/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala File java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala: http://gerrit.cloudera.org:8080/#/c/18159/6/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala@a157 PS6, Line 157: > according to the docs this will be shut down automatically on exit, and cal BTW, does the newly updated code still works fine with Spark 2 (if it have to run with that version at all)? It would be great to clarify -- I guess adding explicit call to shutdown() was not just a random thing. -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 6 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan Gerrit-Comment-Date: Tue, 25 Jan 2022 06:01:17 + Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18159 ) Change subject: [java] Upgrade dependencies .. Patch Set 10: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/18159/7//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/18159/7//COMMIT_MSG@42 PS7, Line 42: 30.1.-jre -> 31.0.1-jre > It's 30 -> 31. Guava is here: https://gerrit.cloudera.org/c/18159/7/java/bu Whoops, indeed -- somehow I missed the major version bump. Thanks for pointing this out. -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 10 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan Gerrit-Comment-Date: Tue, 25 Jan 2022 03:33:37 + Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/18159 ) Change subject: [java] Upgrade dependencies .. Patch Set 10: Verified+1 Unrelated flaky test -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 10 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan Gerrit-Comment-Date: Mon, 24 Jan 2022 18:14:30 + Gerrit-HasComments: No
[kudu-CR] [java] Upgrade dependencies
Attila Bukor has removed a vote on this change. Change subject: [java] Upgrade dependencies .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: deleteVote Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 10 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan
[kudu-CR] [java] Upgrade dependencies
Zoltan Chovan has posted comments on this change. ( http://gerrit.cloudera.org:8080/18159 ) Change subject: [java] Upgrade dependencies .. Patch Set 10: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 10 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan Gerrit-Comment-Date: Mon, 24 Jan 2022 16:14:45 + Gerrit-HasComments: No
[kudu-CR] [java] Upgrade dependencies
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/18159 ) Change subject: [java] Upgrade dependencies .. Patch Set 10: (2 comments) > Patch Set 7: > > (2 comments) > > It seems Java LINT isn't yet happy, complaining about > META-INF/services/reactor.blockhound.integration.BlockHoundIntegration: > http://jenkins.kudu.apache.org/job/kudu-gerrit/24974/BUILD_TYPE=LINT/console I couldn't figure out what changed, it's coming from Netty, but this file has been there for ages. http://gerrit.cloudera.org:8080/#/c/18159/7//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/18159/7//COMMIT_MSG@42 PS7, Line 42: 30.1.-jre -> 31.0.1-jre > This looks rather like a downgrade :) BTW, I didn't find guava version cha It's 30 -> 31. Guava is here: https://gerrit.cloudera.org/c/18159/7/java/buildSrc/build.gradle#41 http://gerrit.cloudera.org:8080/#/c/18159/7/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala File java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala: http://gerrit.cloudera.org:8080/#/c/18159/7/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala@142 PS7, Line 142: val parallelTables = options.tables.par > Since no clean-up is needed per https://gerrit.cloudera.org/#/c/18159/6/jav Done -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 10 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan Gerrit-Comment-Date: Mon, 24 Jan 2022 16:09:05 + Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Hello Zoltan Chovan, Alexey Serbin, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/18159 to look at the new patch set (#10). Change subject: [java] Upgrade dependencies .. [java] Upgrade dependencies Updgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito - 3.8.0 -> 4.2.0 Minor version upgrades: - commons - 2.8.0 -> 2.11.0 - jmh - 1.28 -> 1.34 - micrometer - 1.6.5 -> 1.8.2 - protobuf - 3.15.6 -> 3.19.3 Maintenance version upgrades: - clojure - 1.10.1 -> 1.10.3 - clojure tools cli - 1.0.194 -> 1.0.206 - hadoop - 3.3.0 -> 3.3.1 - jetty - 9.4.38.v20210224 -> 9.4.44.v20210927 - netty - 4.1.65.Final -> 4.1.73.Final - scala -> 2.12.13 -> 2.12.15 - scalatest - 3.2.6 -> 3.2.10 - slf4j - 1.7.30 -> 1.7.33 - spark -> 3.1.1 -> 3.1.2 - spark2 - 2.4.7 -> 2.4.8 Gradle upgrades: - gradle - 6.8.3 -> 6.9.2 - gradle-versions-plugin - 0.38.0 -> 0.41.0 - osdetector-gradle-plugin - 1.6.2 -> 1.7.0 - protobuf-gradle-plugin - 0.8.15 -> 0.8.18 - nebula-clojure-plugin - 9.4.3 -> 10.1.1 - propdeps-plugin - 0.0.9.RELEASE -> 0.0.10.RELEASE - gradle-errorprone-plugin - 1.3.0 -> 2.0.2 - gradle-animalsniffer-plugin - 1.5.3 -> 1.5.4 - gson - 2.8.6 -> 2.8.9 - guava - 30.1.-jre -> 31.0.1-jre Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 --- M build-support/verify_jars.pl M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradlew M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduRestore.scala M java/kudu-client/src/main/java/org/apache/kudu/client/RpcProxy.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduClient.java M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/DefaultSource.scala M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/package.scala 10 files changed, 43 insertions(+), 42 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/59/18159/10 -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 10 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan
[kudu-CR] [java] Upgrade dependencies
Hello Zoltan Chovan, Alexey Serbin, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/18159 to look at the new patch set (#9). Change subject: [java] Upgrade dependencies .. [java] Upgrade dependencies Updgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito - 3.8.0 -> 4.2.0 Minor version upgrades: - commons - 2.8.0 -> 2.11.0 - jmh - 1.28 -> 1.34 - micrometer - 1.6.5 -> 1.8.2 - protobuf - 3.15.6 -> 3.19.3 Maintenance version upgrades: - clojure - 1.10.1 -> 1.10.3 - clojure tools cli - 1.0.194 -> 1.0.206 - hadoop - 3.3.0 -> 3.3.1 - jetty - 9.4.38.v20210224 -> 9.4.44.v20210927 - netty - 4.1.65.Final -> 4.1.73.Final - scala -> 2.12.13 -> 2.12.15 - scalatest - 3.2.6 -> 3.2.10 - slf4j - 1.7.30 -> 1.7.33 - spark -> 3.1.1 -> 3.1.2 - spark2 - 2.4.7 -> 2.4.8 Gradle upgrades: - gradle - 6.8.3 -> 6.9.2 - gradle-versions-plugin - 0.38.0 -> 0.41.0 - osdetector-gradle-plugin - 1.6.2 -> 1.7.0 - protobuf-gradle-plugin - 0.8.15 -> 0.8.18 - nebula-clojure-plugin - 9.4.3 -> 10.1.1 - propdeps-plugin - 0.0.9.RELEASE -> 0.0.10.RELEASE - gradle-errorprone-plugin - 1.3.0 -> 2.0.2 - gradle-animalsniffer-plugin - 1.5.3 -> 1.5.4 - gson - 2.8.6 -> 2.8.9 - guava - 30.1.-jre -> 31.0.1-jre Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradlew M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduRestore.scala M java/kudu-client/src/main/java/org/apache/kudu/client/RpcProxy.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduClient.java M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/DefaultSource.scala M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/package.scala 9 files changed, 40 insertions(+), 41 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/59/18159/9 -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 9 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan
[kudu-CR] [java] Upgrade dependencies
Hello Zoltan Chovan, Alexey Serbin, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/18159 to look at the new patch set (#8). Change subject: [java] Upgrade dependencies .. [java] Upgrade dependencies Updgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito - 3.8.0 -> 4.2.0 Minor version upgrades: - commons - 2.8.0 -> 2.11.0 - jmh - 1.28 -> 1.34 - micrometer - 1.6.5 -> 1.8.2 - protobuf - 3.15.6 -> 3.19.3 Maintenance version upgrades: - clojure - 1.10.1 -> 1.10.3 - clojure tools cli - 1.0.194 -> 1.0.206 - hadoop - 3.3.0 -> 3.3.1 - jetty - 9.4.38.v20210224 -> 9.4.44.v20210927 - netty - 4.1.65.Final -> 4.1.73.Final - scala -> 2.12.13 -> 2.12.15 - scalatest - 3.2.6 -> 3.2.10 - slf4j - 1.7.30 -> 1.7.33 - spark -> 3.1.1 -> 3.1.2 - spark2 - 2.4.7 -> 2.4.8 Gradle upgrades: - gradle - 6.8.3 -> 6.9.2 - gradle-versions-plugin - 0.38.0 -> 0.41.0 - osdetector-gradle-plugin - 1.6.2 -> 1.7.0 - protobuf-gradle-plugin - 0.8.15 -> 0.8.18 - nebula-clojure-plugin - 9.4.3 -> 10.1.1 - propdeps-plugin - 0.0.9.RELEASE -> 0.0.10.RELEASE - gradle-errorprone-plugin - 1.3.0 -> 2.0.2 - gradle-animalsniffer-plugin - 1.5.3 -> 1.5.4 - gson - 2.8.6 -> 2.8.9 - guava - 30.1.-jre -> 31.0.1-jre Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradlew M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduRestore.scala M java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduClient.java M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/DefaultSource.scala 7 files changed, 38 insertions(+), 38 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/59/18159/8 -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 8 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan
[kudu-CR] [java] Upgrade dependencies
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18159 ) Change subject: [java] Upgrade dependencies .. Patch Set 7: (2 comments) It seems Java LINT isn't yet happy, complaining about META-INF/services/reactor.blockhound.integration.BlockHoundIntegration: http://jenkins.kudu.apache.org/job/kudu-gerrit/24974/BUILD_TYPE=LINT/console http://gerrit.cloudera.org:8080/#/c/18159/7//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/18159/7//COMMIT_MSG@42 PS7, Line 42: 30.1.-jre -> 31.0.1-jre This looks rather like a downgrade :) BTW, I didn't find guava version changed in gradle files in PS7 -- is it missing somehow? http://gerrit.cloudera.org:8080/#/c/18159/7/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala File java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala: http://gerrit.cloudera.org:8080/#/c/18159/7/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala@142 PS7, Line 142: val pool = new ForkJoinPool(options.numParallelBackups) // Need a clean-up reference. Since no clean-up is needed per https://gerrit.cloudera.org/#/c/18159/6/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala@a157, maybe it's time to drop the 'pool' variable and join create the pool in-place as an argument for ForkJoinTaskSupport() below? -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 7 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan Gerrit-Comment-Date: Thu, 20 Jan 2022 16:35:56 + Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Hello Zoltan Chovan, Alexey Serbin, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/18159 to look at the new patch set (#7). Change subject: [java] Upgrade dependencies .. [java] Upgrade dependencies Updgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito - 3.8.0 -> 4.2.0 Minor version upgrades: - commons - 2.8.0 -> 2.11.0 - jmh - 1.28 -> 1.34 - micrometer - 1.6.5 -> 1.8.2 - protobuf - 3.15.6 -> 3.19.3 Maintenance version upgrades: - clojure - 1.10.1 -> 1.10.3 - clojure tools cli - 1.0.194 -> 1.0.206 - hadoop - 3.3.0 -> 3.3.1 - jetty - 9.4.38.v20210224 -> 9.4.44.v20210927 - netty - 4.1.65.Final -> 4.1.73.Final - scala -> 2.12.13 -> 2.12.15 - scalatest - 3.2.6 -> 3.2.10 - slf4j - 1.7.30 -> 1.7.33 - spark -> 3.1.1 -> 3.1.2 - spark2 - 2.4.7 -> 2.4.8 Gradle upgrades: - gradle - 6.8.3 -> 6.9.2 - gradle-versions-plugin - 0.38.0 -> 0.41.0 - osdetector-gradle-plugin - 1.6.2 -> 1.7.0 - protobuf-gradle-plugin - 0.8.15 -> 0.8.18 - nebula-clojure-plugin - 9.4.3 -> 10.1.1 - propdeps-plugin - 0.0.9.RELEASE -> 0.0.10.RELEASE - gradle-errorprone-plugin - 1.3.0 -> 2.0.2 - gradle-animalsniffer-plugin - 1.5.3 -> 1.5.4 - gson - 2.8.6 -> 2.8.9 - guava - 30.1.-jre -> 31.0.1-jre Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradlew M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduRestore.scala M java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduClient.java 6 files changed, 30 insertions(+), 31 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/59/18159/7 -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 7 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan
[kudu-CR] [java] Upgrade dependencies
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/18159 ) Change subject: [java] Upgrade dependencies .. Patch Set 7: (3 comments) http://gerrit.cloudera.org:8080/#/c/18159/6//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/18159/6//COMMIT_MSG@26 PS6, Line 26: 1 > this version was: Done http://gerrit.cloudera.org:8080/#/c/18159/6/java/gradlew File java/gradlew: http://gerrit.cloudera.org:8080/#/c/18159/6/java/gradlew@87 PS6, Line 87: 8.3 > shouldn't this be 6.9.2? seems this is always one step behind, not sure why. The file is autogenerated by gradle. http://gerrit.cloudera.org:8080/#/c/18159/6/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala File java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala: http://gerrit.cloudera.org:8080/#/c/18159/6/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala@a157 PS6, Line 157: > why is this not necessary? according to the docs this will be shut down automatically on exit, and calling filter on backupResults not works in 3.1.2. -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 7 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan Gerrit-Comment-Date: Thu, 20 Jan 2022 16:22:47 + Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Zoltan Chovan has posted comments on this change. ( http://gerrit.cloudera.org:8080/18159 ) Change subject: [java] Upgrade dependencies .. Patch Set 6: (3 comments) http://gerrit.cloudera.org:8080/#/c/18159/6//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/18159/6//COMMIT_MSG@26 PS6, Line 26: 1 this version was: 2.12.13 http://gerrit.cloudera.org:8080/#/c/18159/6/java/gradlew File java/gradlew: http://gerrit.cloudera.org:8080/#/c/18159/6/java/gradlew@87 PS6, Line 87: 8.3 shouldn't this be 6.9.2? http://gerrit.cloudera.org:8080/#/c/18159/6/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala File java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala: http://gerrit.cloudera.org:8080/#/c/18159/6/java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala@a157 PS6, Line 157: why is this not necessary? -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 6 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan Gerrit-Comment-Date: Thu, 20 Jan 2022 16:09:16 + Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Hello Zoltan Chovan, Alexey Serbin, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/18159 to look at the new patch set (#6). Change subject: [java] Upgrade dependencies .. [java] Upgrade dependencies Updgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito - 3.8.0 -> 4.2.0 Minor version upgrades: - commons - 2.8.0 -> 2.11.0 - jmh - 1.28 -> 1.34 - micrometer - 1.6.5 -> 1.8.2 - protobuf - 3.15.6 -> 3.19.3 Maintenance version upgrades: - clojure - 1.10.1 -> 1.10.3 - clojure tools cli - 1.0.194 -> 1.0.206 - hadoop - 3.3.0 -> 3.3.1 - jetty - 9.4.38.v20210224 -> 9.4.44.v20210927 - netty - 4.1.65.Final -> 4.1.73.Final - scala -> 2.12.1 -> 2.12.15 - scalatest - 3.2.6 -> 3.2.10 - slf4j - 1.7.30 -> 1.7.33 - spark -> 3.1.1 -> 3.1.2 - spark2 - 2.4.7 -> 2.4.8 Gradle upgrades: - gradle - 6.8.3 -> 6.9.2 - gradle-versions-plugin - 0.38.0 -> 0.41.0 - osdetector-gradle-plugin - 1.6.2 -> 1.7.0 - protobuf-gradle-plugin - 0.8.15 -> 0.8.18 - nebula-clojure-plugin - 9.4.3 -> 10.1.1 - propdeps-plugin - 0.0.9.RELEASE -> 0.0.10.RELEASE - gradle-errorprone-plugin - 1.3.0 -> 2.0.2 - gradle-animalsniffer-plugin - 1.5.3 -> 1.5.4 - gson - 2.8.6 -> 2.8.9 - guava - 30.1.-jre -> 31.0.1-jre Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradlew M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduRestore.scala M java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduClient.java 6 files changed, 30 insertions(+), 31 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/59/18159/6 -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 6 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan
[kudu-CR] [java] Upgrade dependencies
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/18159 ) Change subject: [java] Upgrade dependencies .. Patch Set 4: I upgraded Spark only to 3.1.2 instead of 3.2.0 due to SPARK-37690 -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 4 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Thu, 20 Jan 2022 14:51:44 + Gerrit-HasComments: No
[kudu-CR] [java] Upgrade dependencies
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/18159 to look at the new patch set (#5). Change subject: [java] Upgrade dependencies .. [java] Upgrade dependencies Updgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito - 3.8.0 -> 4.2.0 Minor version upgrades: - commons - 2.8.0 -> 2.11.0 - jmh - 1.28 -> 1.34 - micrometer - 1.6.5 -> 1.8.2 - protobuf - 3.15.6 -> 3.19.3 Maintenance version upgrades: - clojure - 1.10.1 -> 1.10.3 - clojure tools cli - 1.0.194 -> 1.0.206 - hadoop - 3.3.0 -> 3.3.1 - jetty - 9.4.38.v20210224 -> 9.4.44.v20210927 - netty - 4.1.65.Final -> 4.1.73.Final - scala -> 2.12.1 -> 2.12.15 - scalatest - 3.2.6 -> 3.2.10 - slf4j - 1.7.30 -> 1.7.33 - spark -> 3.1.1 -> 3.1.2 - spark2 - 2.4.7 -> 2.4.8 Gradle upgrades: - gradle - 6.8.3 -> 6.9.2 - gradle-versions-plugin - 0.38.0 -> 0.41.0 - osdetector-gradle-plugin - 1.6.2 -> 1.7.0 - protobuf-gradle-plugin - 0.8.15 -> 0.8.18 - nebula-clojure-plugin - 9.4.3 -> 10.1.1 - propdeps-plugin - 0.0.9.RELEASE -> 0.0.10.RELEASE - gradle-errorprone-plugin - 1.3.0 -> 2.0.2 - gradle-animalsniffer-plugin - 1.5.3 -> 1.5.4 - gson - 2.8.6 -> 2.8.9 - guava - 30.1.-jre -> 31.0.1-jre Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradlew M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduBackup.scala M java/kudu-backup/src/main/scala/org/apache/kudu/backup/KuduRestore.scala 5 files changed, 26 insertions(+), 28 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/59/18159/5 -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 5 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [java] Upgrade dependencies
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/18159 to look at the new patch set (#4). Change subject: [java] Upgrade dependencies .. [java] Upgrade dependencies Updgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito - 3.8.0 -> 4.2.0 Minor version upgrades: - commons - 2.8.0 -> 2.11.0 - jmh - 1.28 -> 1.34 - micrometer - 1.6.5 -> 1.8.2 - protobuf - 3.15.6 -> 3.19.3 Maintenance version upgrades: - clojure - 1.10.1 -> 1.10.3 - clojure tools cli - 1.0.194 -> 1.0.206 - hadoop - 3.3.0 -> 3.3.1 - jetty - 9.4.38.v20210224 -> 9.4.44.v20210927 - netty - 4.1.65.Final -> 4.1.73.Final - scala -> 2.12.1 -> 2.12.15 - scalatest - 3.2.6 -> 3.2.10 - slf4j - 1.7.30 -> 1.7.33 - spark -> 3.1.1 -> 3.1.2 - spark2 - 2.4.7 -> 2.4.8 Gradle upgrades: - gradle - 6.8.3 -> 6.9.2 - gradle-versions-plugin - 0.38.0 -> 0.41.0 - osdetector-gradle-plugin - 1.6.2 -> 1.7.0 - protobuf-gradle-plugin - 0.8.15 -> 0.8.18 - nebula-clojure-plugin - 9.4.3 -> 10.1.1 - propdeps-plugin - 0.0.9.RELEASE -> 0.0.10.RELEASE - gradle-errorprone-plugin - 1.3.0 -> 2.0.2 - gradle-animalsniffer-plugin - 1.5.3 -> 1.5.4 - gson - 2.8.6 -> 2.8.9 - guava - 30.1.-jre -> 31.0.1-jre Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradlew 3 files changed, 26 insertions(+), 26 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/59/18159/4 -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 4 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [java] Upgrade dependencies
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/18159 ) Change subject: [java] Upgrade dependencies .. Patch Set 3: > Uploaded patch set 3. Ranger can't be upgraded due to RANGER-3108 -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 3 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 19 Jan 2022 15:16:29 + Gerrit-HasComments: No
[kudu-CR] [java] Upgrade dependencies
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/18159 to look at the new patch set (#3). Change subject: [java] Upgrade dependencies .. [java] Upgrade dependencies Updgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito - 3.8.0 -> 4.2.0 Minor version upgrades: - commons - 2.8.0 -> 2.11.0 - jmh - 1.28 -> 1.34 - micrometer - 1.6.5 -> 1.8.2 - protobuf - 3.15.6 -> 3.19.3 - spark -> 3.1.1 -> 3.2.0 Maintenance version upgrades: - clojure - 1.10.1 -> 1.10.3 - clojure tools cli - 1.0.194 -> 1.0.206 - hadoop - 3.3.0 -> 3.3.1 - jetty - 9.4.38.v20210224 -> 9.4.44.v20210927 - netty - 4.1.65.Final -> 4.1.73.Final - scala -> 2.12.1 -> 2.12.15 - scalatest - 3.2.6 -> 3.2.10 - slf4j - 1.7.30 -> 1.7.33 - spark2 - 2.4.7 -> 2.4.8 Gradle upgrades: - gradle - 6.8.3 -> 6.9.2 - gradle-versions-plugin - 0.38.0 -> 0.41.0 - osdetector-gradle-plugin - 1.6.2 -> 1.7.0 - protobuf-gradle-plugin - 0.8.15 -> 0.8.18 - nebula-clojure-plugin - 9.4.3 -> 10.1.1 - propdeps-plugin - 0.0.9.RELEASE -> 0.0.10.RELEASE - gradle-errorprone-plugin - 1.3.0 -> 2.0.2 - gradle-animalsniffer-plugin - 1.5.3 -> 1.5.4 - gson - 2.8.6 -> 2.8.9 - guava - 30.1.-jre -> 31.0.1-jre Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradlew 3 files changed, 26 insertions(+), 26 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/59/18159/3 -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 3 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [java] Upgrade dependencies
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/18159 to look at the new patch set (#2). Change subject: [java] Upgrade dependencies .. [java] Upgrade dependencies Updgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito - 3.8.0 -> 4.2.0 Minor version upgrades: - commons - 2.8.0 -> 2.11.0 - jmh - 1.28 -> 1.34 - micrometer - 1.6.5 -> 1.8.2 - protobuf - 3.15.6 -> 3.19.3 - ranger - 2.1.0 -> 2.2.0 - spark -> 3.1.1 -> 3.2.0 Maintenance version upgrades: - clojure - 1.10.1 -> 1.10.3 - clojure tools cli - 1.0.194 -> 1.0.206 - hadoop - 3.3.0 -> 3.3.1 - jetty - 9.4.38.v20210224 -> 9.4.44.v20210927 - netty - 4.1.65.Final -> 4.1.73.Final - scala -> 2.12.1 -> 2.12.15 - scalatest - 3.2.6 -> 3.2.10 - slf4j - 1.7.30 -> 1.7.33 - spark2 - 2.4.7 -> 2.4.8 Gradle upgrades: - gradle - 6.8.3 -> 6.9.2 - gradle-versions-plugin - 0.38.0 -> 0.41.0 - osdetector-gradle-plugin - 1.6.2 -> 1.7.0 - protobuf-gradle-plugin - 0.8.15 -> 0.8.18 - nebula-clojure-plugin - 9.4.3 -> 10.1.1 - propdeps-plugin - 0.0.9.RELEASE -> 0.0.10.RELEASE - gradle-errorprone-plugin - 1.3.0 -> 2.0.2 - gradle-animalsniffer-plugin - 1.5.3 -> 1.5.4 - gson - 2.8.6 -> 2.8.9 - guava - 30.1.-jre -> 31.0.1-jre Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradlew 3 files changed, 27 insertions(+), 27 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/59/18159/2 -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 2 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [java] Upgrade dependencies
Attila Bukor has uploaded this change for review. ( http://gerrit.cloudera.org:8080/18159 Change subject: [java] Upgrade dependencies .. [java] Upgrade dependencies Updgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito - 3.8.0 -> 4.2.0 Minor version upgrades: - commons - 2.8.0 -> 2.11.0 - error-prone - 2.3.3 -> 2.10.0 - jmh - 1.28 -> 1.34 - micrometer - 1.6.5 -> 1.8.2 - protobuf - 3.15.6 -> 3.19.3 - ranger - 2.1.0 -> 2.2.0 - spark -> 3.1.1 -> 3.2.0 Maintenance version upgrades: - clojure - 1.10.1 -> 1.10.3 - clojure tools cli - 1.0.194 -> 1.0.206 - hadoop - 3.3.0 -> 3.3.1 - jetty - 9.4.38.v20210224 -> 9.4.44.v20210927 - netty - 4.1.65.Final -> 4.1.73.Final - scala -> 2.12.1 -> 2.12.15 - scalatest - 3.2.6 -> 3.2.10 - slf4j - 1.7.30 -> 1.7.33 - spark2 - 2.4.7 -> 2.4.8 Gradle upgrades: - gradle - 6.8.3 -> 6.9.2 - gradle-versions-plugin - 0.38.0 -> 0.41.0 - osdetector-gradle-plugin - 1.6.2 -> 1.7.0 - protobuf-gradle-plugin - 0.8.15 -> 0.8.18 - nebula-clojure-plugin - 9.4.3 -> 10.1.1 - propdeps-plugin - 0.0.9.RELEASE -> 0.0.10.RELEASE - gradle-errorprone-plugin - 1.3.0 -> 2.0.2 - gradle-animalsniffer-plugin - 1.5.3 -> 1.5.4 - gson - 2.8.6 -> 2.8.9 - guava - 30.1.-jre -> 31.0.1-jre Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradlew 3 files changed, 28 insertions(+), 28 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/59/18159/1 -- To view, visit http://gerrit.cloudera.org:8080/18159 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I300af97556a8c12a2e7c0deb2ae723c2ea639518 Gerrit-Change-Number: 18159 Gerrit-PatchSet: 1 Gerrit-Owner: Attila Bukor
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/17206 ) Change subject: [Java] Upgrade dependencies .. Patch Set 2: I see TestKuduBackup failed in a couple of the builds. I can follow up on Monday. -- To view, visit http://gerrit.cloudera.org:8080/17206 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie52174f558ae9ce3075f377303a5d8ac23ff57ec Gerrit-Change-Number: 17206 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Sun, 21 Mar 2021 04:07:57 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/17206 ) Change subject: [Java] Upgrade dependencies .. Patch Set 2: Ah oops. Seems there's some flakiness in the Java tests that may actually be related. Digging in. -- To view, visit http://gerrit.cloudera.org:8080/17206 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie52174f558ae9ce3075f377303a5d8ac23ff57ec Gerrit-Change-Number: 17206 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Sun, 21 Mar 2021 03:22:22 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17206 ) Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - jetty 9.4.35.v20201120 -> 9.4.38.v20210224 - jmh 1.27 -> 1.28 - mockito 3.6.28 -> 3.8.0 - protobuf 3.14.0 -> 3.15.6 - spark 3.0.1 -> 3.1.1 Maintenance version upgrades: - junit 4.13.1 -> 4.13.2 - log4j 2.14.0 -> 2.14.1 - micrometer 1.6.2 -> 1.6.5 - netty 4.1.56.Final -> 4.1.60.Final - scala 2.13.12 -> 2.12.13 - scalatest 3.2.3 -> 3.2.6 - scopt 4.0.0 -> 4.0.1 Gradle upgrades: - gradle 6.7.1 -> 6.8.3 - gradle-versions-plugin 0.36.0 -> 0.38.0 - gradle-jacoco-log 1.2.3 -> 1.2.4 - protobuf-gradle-plugin 0.8.14 -> 0.8.15 - spotbugs-gradle-plugin 4.6.0 -> 4.7.0 - gradle-animalsniffer-plugin 1.5.2 -> 1.5.3 - jmh-gradle-plugin 0.5.2 -> 0.5.3 Change-Id: Ie52174f558ae9ce3075f377303a5d8ac23ff57ec Reviewed-on: http://gerrit.cloudera.org:8080/17206 Reviewed-by: Andrew Wong Tested-by: Andrew Wong Reviewed-by: Alexey Serbin --- M java/buildSrc/build.gradle M java/gradle.properties M java/gradle/dependencies.gradle M java/gradle/wrapper/gradle-wrapper.properties M java/gradlew 5 files changed, 23 insertions(+), 21 deletions(-) Approvals: Andrew Wong: Looks good to me, approved; Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/17206 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie52174f558ae9ce3075f377303a5d8ac23ff57ec Gerrit-Change-Number: 17206 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17206 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/17206 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie52174f558ae9ce3075f377303a5d8ac23ff57ec Gerrit-Change-Number: 17206 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 19 Mar 2021 19:51:06 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/17206 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: Verified+1 Code-Review+2 Test failure is unrelated. -- To view, visit http://gerrit.cloudera.org:8080/17206 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie52174f558ae9ce3075f377303a5d8ac23ff57ec Gerrit-Change-Number: 17206 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 19 Mar 2021 18:22:44 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Andrew Wong has removed a vote on this change. Change subject: [Java] Upgrade dependencies .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/17206 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: deleteVote Gerrit-Change-Id: Ie52174f558ae9ce3075f377303a5d8ac23ff57ec Gerrit-Change-Number: 17206 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17206 Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - jetty 9.4.35.v20201120 -> 9.4.38.v20210224 - jmh 1.27 -> 1.28 - mockito 3.6.28 -> 3.8.0 - protobuf 3.14.0 -> 3.15.6 - spark 3.0.1 -> 3.1.1 Maintenance version upgrades: - junit 4.13.1 -> 4.13.2 - log4j 2.14.0 -> 2.14.1 - micrometer 1.6.2 -> 1.6.5 - netty 4.1.56.Final -> 4.1.60.Final - scala 2.13.12 -> 2.12.13 - scalatest 3.2.3 -> 3.2.6 - scopt 4.0.0 -> 4.0.1 Gradle upgrades: - gradle 6.7.1 -> 6.8.3 - gradle-versions-plugin 0.36.0 -> 0.38.0 - gradle-jacoco-log 1.2.3 -> 1.2.4 - protobuf-gradle-plugin 0.8.14 -> 0.8.15 - spotbugs-gradle-plugin 4.6.0 -> 4.7.0 - gradle-animalsniffer-plugin 1.5.2 -> 1.5.3 - jmh-gradle-plugin 0.5.2 -> 0.5.3 Change-Id: Ie52174f558ae9ce3075f377303a5d8ac23ff57ec --- M java/buildSrc/build.gradle M java/gradle.properties M java/gradle/dependencies.gradle M java/gradle/wrapper/gradle-wrapper.properties M java/gradlew 5 files changed, 23 insertions(+), 21 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/06/17206/1 -- To view, visit http://gerrit.cloudera.org:8080/17206 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ie52174f558ae9ce3075f377303a5d8ac23ff57ec Gerrit-Change-Number: 17206 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16874 ) Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Major version upgrades: - guava 29.0-jre -> 30.1-jre - scopt 3.7.1 -> 4.0.0 Minor version upgrades: - jmh 1.26 -> 1.27 - log4j 2.13.3 -> 2.14.0 - micrometer 1.5.5 -> 1.6.2 - mockito 3.5.13 -> 3.6.28 - protobuf 3.13.0 -> 3.14.0 - yetus 0.12.0 -> 0.13.0 Maintenance version upgrades: - jetty 9.4.32.v20200930 -> 9.4.35.v20201120 - netty 4.1.52.Final -> 4.1.56.Final - scalatest 3.2.2 -> 3.2.3 Gradle upgrades: - gradle 6.6.1 -> 6.7.1 - gradle-versions-plugin 0.33.0 -> 0.36.0 - gradle-protobuf-plugin 0.8.13 -> 0.8.14 - nebula-clojure-plugin 9.4.1 -> 9.4.2 - spotbugs-gradle-plugin 4.5.0 -> 4.6.0 - gradle-error-prone-plugin 1.2.1 -> 1.3.0 - gradle-animal-sniffer-plugin 1.5.1 -> 1.5.2 - jmh-gradle-plugin 0.5.1 -> 0.5.2 Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 Reviewed-on: http://gerrit.cloudera.org:8080/16874 Reviewed-by: Attila Bukor Tested-by: Kudu Jenkins --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradle/wrapper/gradle-wrapper.properties M java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestOptions.scala M java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiator.java 5 files changed, 27 insertions(+), 26 deletions(-) Approvals: Attila Bukor: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/16874 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 Gerrit-Change-Number: 16874 Gerrit-PatchSet: 4 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16874 ) Change subject: [Java] Upgrade dependencies .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/16874 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 Gerrit-Change-Number: 16874 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 05 Jan 2021 15:33:27 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Hello Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16874 to look at the new patch set (#3). Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Major version upgrades: - guava 29.0-jre -> 30.1-jre - scopt 3.7.1 -> 4.0.0 Minor version upgrades: - jmh 1.26 -> 1.27 - log4j 2.13.3 -> 2.14.0 - micrometer 1.5.5 -> 1.6.2 - mockito 3.5.13 -> 3.6.28 - protobuf 3.13.0 -> 3.14.0 - yetus 0.12.0 -> 0.13.0 Maintenance version upgrades: - jetty 9.4.32.v20200930 -> 9.4.35.v20201120 - netty 4.1.52.Final -> 4.1.56.Final - scalatest 3.2.2 -> 3.2.3 Gradle upgrades: - gradle 6.6.1 -> 6.7.1 - gradle-versions-plugin 0.33.0 -> 0.36.0 - gradle-protobuf-plugin 0.8.13 -> 0.8.14 - nebula-clojure-plugin 9.4.1 -> 9.4.2 - spotbugs-gradle-plugin 4.5.0 -> 4.6.0 - gradle-error-prone-plugin 1.2.1 -> 1.3.0 - gradle-animal-sniffer-plugin 1.5.1 -> 1.5.2 - jmh-gradle-plugin 0.5.1 -> 0.5.2 Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradle/wrapper/gradle-wrapper.properties M java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestOptions.scala M java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiator.java 5 files changed, 27 insertions(+), 26 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/74/16874/3 -- To view, visit http://gerrit.cloudera.org:8080/16874 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 Gerrit-Change-Number: 16874 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16874 ) Change subject: [Java] Upgrade dependencies .. Patch Set 2: maybe we should upgrade the Gradle wrapper to 6.7.1 as well? -- To view, visit http://gerrit.cloudera.org:8080/16874 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 Gerrit-Change-Number: 16874 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 05 Jan 2021 15:19:21 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16874 ) Change subject: [Java] Upgrade dependencies .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/16874/2/java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiator.java File java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiator.java: http://gerrit.cloudera.org:8080/#/c/16874/2/java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiator.java@427 PS2, Line 427: while (((ByteBuf) embedder.outboundMessages().peek()).readableBytes() == 0) { : embedder.readOutbound(); : } Just curious: is this change due to the fact that with the newer version of some related packages now there are multiple empty messages, while with prior versions there could be at most one empty message? -- To view, visit http://gerrit.cloudera.org:8080/16874 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 Gerrit-Change-Number: 16874 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 04 Jan 2021 22:44:33 + Gerrit-HasComments: Yes
[kudu-CR] [Java] Upgrade dependencies
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16874 ) Change subject: [Java] Upgrade dependencies .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/16874 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 Gerrit-Change-Number: 16874 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 04 Jan 2021 18:54:32 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16874 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/16874/1/java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiator.java File java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiator.java: http://gerrit.cloudera.org:8080/#/c/16874/1/java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiator.java@425 PS1, Line 425: an > nit: drop Done -- To view, visit http://gerrit.cloudera.org:8080/16874 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 Gerrit-Change-Number: 16874 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 04 Jan 2021 18:38:04 + Gerrit-HasComments: Yes
[kudu-CR] [Java] Upgrade dependencies
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16874 to look at the new patch set (#2). Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Major version upgrades: - guava 29.0-jre -> 30.1-jre - scopt 3.7.1 -> 4.0.0 Minor version upgrades: - jmh 1.26 -> 1.27 - log4j 2.13.3 -> 2.14.0 - micrometer 1.5.5 -> 1.6.2 - mockito 3.5.13 -> 3.6.28 - protobuf 3.13.0 -> 3.14.0 - yetus 0.12.0 -> 0.13.0 Maintenance version upgrades: - jetty 9.4.32.v20200930 -> 9.4.35.v20201120 - netty 4.1.52.Final -> 4.1.56.Final - scalatest 3.2.2 -> 3.2.3 Gradle upgrades: - gradle 6.6.1 -> 6.7.1 - gradle-versions-plugin 0.33.0 -> 0.36.0 - gradle-protobuf-plugin 0.8.13 -> 0.8.14 - nebula-clojure-plugin 9.4.1 -> 9.4.2 - spotbugs-gradle-plugin 4.5.0 -> 4.6.0 - gradle-error-prone-plugin 1.2.1 -> 1.3.0 - gradle-animal-sniffer-plugin 1.5.1 -> 1.5.2 - jmh-gradle-plugin 0.5.1 -> 0.5.2 Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestOptions.scala M java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiator.java 4 files changed, 26 insertions(+), 25 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/74/16874/2 -- To view, visit http://gerrit.cloudera.org:8080/16874 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 Gerrit-Change-Number: 16874 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16874 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/16874 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 Gerrit-Change-Number: 16874 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 04 Jan 2021 14:31:02 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16874 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/16874/1/java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiator.java File java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiator.java: http://gerrit.cloudera.org:8080/#/c/16874/1/java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiator.java@425 PS1, Line 425: an nit: drop -- To view, visit http://gerrit.cloudera.org:8080/16874 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 Gerrit-Change-Number: 16874 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 18 Dec 2020 01:05:02 + Gerrit-HasComments: Yes
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16874 Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Major version upgrades: - guava 29.0-jre -> 30.0-jre - scopt 3.7.1 -> 4.0.0 Minor version upgrades: - jmh 1.26 -> 1.27 - log4j 2.13.3 -> 2.14.0 - micrometer 1.5.5 -> 1.6.2 - mockito 3.5.13 -> 3.6.28 - protobuf 3.13.0 -> 3.14.0 - yetus 0.12.0 -> 0.13.0 Maintenance version upgrades: - jetty 9.4.32.v20200930 -> 9.4.35.v20201120 - netty 4.1.52.Final -> 4.1.55.Final - scalatest 3.2.2 -> 3.2.3 Gradle upgrades: - gradle 6.6.1 -> 6.7.1 - gradle-versions-plugin 0.33.0 -> 0.36.0 - gradle-protobuf-plugin 0.8.13 -> 0.8.14 - nebula-clojure-plugin 9.4.1 -> 9.4.2 - spotbugs-gradle-plugin 4.5.0 -> 4.6.0 - gradle-error-prone-plugin 1.2.1 -> 1.3.0 - gradle-animal-sniffer-plugin 1.5.1 -> 1.5.2 - jmh-gradle-plugin 0.5.1 -> 0.5.2 Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestOptions.scala M java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiator.java 4 files changed, 26 insertions(+), 25 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/74/16874/1 -- To view, visit http://gerrit.cloudera.org:8080/16874 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ib4b319a67f629bcb60ae54187cee1c3257d6e1a2 Gerrit-Change-Number: 16874 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16583 ) Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - jmh 1.25.2 -> 1.26 Maintenance version upgrades: - httpClient 4.5.12 -> 4.5.13 - jetty 9.4.31.v20200723 -> 9.4.32.v20200930 - junit 4.13 -> 4.13.1 - micrometer 1.5.4 -> 1.5.5 - mockito 3.5.10 -> 3.5.13 Gradle upgrades: - shadow 6.0.0 -> 6.1.0 Change-Id: If7febccddbf080520ebdf6b4b71cc1571df1927b Reviewed-on: http://gerrit.cloudera.org:8080/16583 Tested-by: Grant Henke Reviewed-by: Andrew Wong --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle 2 files changed, 7 insertions(+), 7 deletions(-) Approvals: Grant Henke: Verified Andrew Wong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/16583 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If7febccddbf080520ebdf6b4b71cc1571df1927b Gerrit-Change-Number: 16583 Gerrit-PatchSet: 4 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Hao Hao Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16583 ) Change subject: [Java] Upgrade dependencies .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/16583 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If7febccddbf080520ebdf6b4b71cc1571df1927b Gerrit-Change-Number: 16583 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Hao Hao Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 13 Oct 2020 19:37:53 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16583 ) Change subject: [Java] Upgrade dependencies .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/16583 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If7febccddbf080520ebdf6b4b71cc1571df1927b Gerrit-Change-Number: 16583 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Hao Hao Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 13 Oct 2020 14:45:35 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has removed a vote on this change. Change subject: [Java] Upgrade dependencies .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/16583 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: deleteVote Gerrit-Change-Id: If7febccddbf080520ebdf6b4b71cc1571df1927b Gerrit-Change-Number: 16583 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Hao Hao Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16583 Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - jmh 1.25.2 -> 1.26 Maintenance version upgrades: - httpClient 4.5.12 -> 4.5.13 - jetty 9.4.31.v20200723 -> 9.4.32.v20200930 - junit 4.13 -> 4.13.1 - micrometer 1.5.4 -> 1.5.5 - mockito 3.5.10 -> 3.5.13 Gradle upgrades: - shadow 6.0.0 -> 6.1.0 Change-Id: If7febccddbf080520ebdf6b4b71cc1571df1927b --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle 2 files changed, 7 insertions(+), 7 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/83/16583/1 -- To view, visit http://gerrit.cloudera.org:8080/16583 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: If7febccddbf080520ebdf6b4b71cc1571df1927b Gerrit-Change-Number: 16583 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16455 ) Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - avro 1.8.2 -> 1.10.0 - clojure 1.9.0 -> 1.10.1 - commonsIo 2.6 -> 2.8.0 - hadoop 3.2.1 -> 3.3.0 - jmh 1.23 -> 1.25.2 - log4j 2.11.2 -> 2.13.3 - mockito 3.3.3 -> 3.5.10 - protobuf 3.11.4 -> 3.13.0 - ranger 2.0.0 -> 2.1.0 - scalatest 3.0.8 -> 3.2.2 Maintenance version upgrades: - jetty 9.4.28.v20200408 -> 9.4.31.v20200723 - micrometer 1.5.1 -> 1.5.4 - netty 4.1.49.Final -> 4.1.52.Final - parquet 1.11.0 -> 1.11.1 - spark 2.4.5 -> 2.4.7 Gradle upgrades: - gradle 6.2.2 -> 6.6.1 - checkstyle 8.29 -> 8.36.1 - clojureToolsCli 0.4.1 -> 1.0.194 - spotBugs 4.0.0-beta1 -> 4.1.1 - gradle-versions-plugin 0.28.0 -> 0.33.0 - shadow 5.2.0 -> 6.0.0 - protobuf-gradle-plugin 0.8.12 -> 0.8.13 - nebula-clojure-plugin 9.4.0 -> 9.4.1 - spotbugs-gradle-plugin 4.0.2 -> 4.5.0 - gradle-errorprone-plugin 1.1.1 -> 1.2.1 - gradle-animalsniffer-plugin 1.5.0 -> 1.5.1 - gradle-scalafmt 1.7.2 -> 1.14.0 - jmh-gradle-plugin 0.5.0 -> 0.5.1 Change-Id: Icc61547b6844893085de8c0f7edee50d84a248cf Reviewed-on: http://gerrit.cloudera.org:8080/16455 Reviewed-by: Andrew Wong Tested-by: Kudu Jenkins --- M java/buildSrc/build.gradle M java/config/spotbugs/excludeFilter.xml M java/gradle/dependencies.gradle M java/gradle/quality.gradle M java/kudu-backup-common/build.gradle M java/kudu-backup/build.gradle M java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestKuduBackup.scala M java/kudu-client-tools/build.gradle M java/kudu-client/build.gradle M java/kudu-hive/build.gradle M java/kudu-mapreduce/build.gradle M java/kudu-spark-tools/build.gradle M java/kudu-spark-tools/src/test/scala/org/apache/kudu/spark/tools/TestImportExportFiles.scala M java/kudu-spark/build.gradle M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduContextTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduRDDTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduTestSuite.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/SparkSQLTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/StreamingTest.scala M java/kudu-subprocess/build.gradle M java/kudu-test-utils/build.gradle 22 files changed, 79 insertions(+), 59 deletions(-) Approvals: Andrew Wong: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/16455 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Icc61547b6844893085de8c0f7edee50d84a248cf Gerrit-Change-Number: 16455 Gerrit-PatchSet: 4 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16455 ) Change subject: [Java] Upgrade dependencies .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/16455 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Icc61547b6844893085de8c0f7edee50d84a248cf Gerrit-Change-Number: 16455 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 15 Sep 2020 20:02:50 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16455 to look at the new patch set (#3). Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - avro 1.8.2 -> 1.10.0 - clojure 1.9.0 -> 1.10.1 - commonsIo 2.6 -> 2.8.0 - hadoop 3.2.1 -> 3.3.0 - jmh 1.23 -> 1.25.2 - log4j 2.11.2 -> 2.13.3 - mockito 3.3.3 -> 3.5.10 - protobuf 3.11.4 -> 3.13.0 - ranger 2.0.0 -> 2.1.0 - scalatest 3.0.8 -> 3.2.2 Maintenance version upgrades: - jetty 9.4.28.v20200408 -> 9.4.31.v20200723 - micrometer 1.5.1 -> 1.5.4 - netty 4.1.49.Final -> 4.1.52.Final - parquet 1.11.0 -> 1.11.1 - spark 2.4.5 -> 2.4.7 Gradle upgrades: - gradle 6.2.2 -> 6.6.1 - checkstyle 8.29 -> 8.36.1 - clojureToolsCli 0.4.1 -> 1.0.194 - spotBugs 4.0.0-beta1 -> 4.1.1 - gradle-versions-plugin 0.28.0 -> 0.33.0 - shadow 5.2.0 -> 6.0.0 - protobuf-gradle-plugin 0.8.12 -> 0.8.13 - nebula-clojure-plugin 9.4.0 -> 9.4.1 - spotbugs-gradle-plugin 4.0.2 -> 4.5.0 - gradle-errorprone-plugin 1.1.1 -> 1.2.1 - gradle-animalsniffer-plugin 1.5.0 -> 1.5.1 - gradle-scalafmt 1.7.2 -> 1.14.0 - jmh-gradle-plugin 0.5.0 -> 0.5.1 Change-Id: Icc61547b6844893085de8c0f7edee50d84a248cf --- M java/buildSrc/build.gradle M java/config/spotbugs/excludeFilter.xml M java/gradle/dependencies.gradle M java/gradle/quality.gradle M java/kudu-backup-common/build.gradle M java/kudu-backup/build.gradle M java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestKuduBackup.scala M java/kudu-client-tools/build.gradle M java/kudu-client/build.gradle M java/kudu-hive/build.gradle M java/kudu-mapreduce/build.gradle M java/kudu-spark-tools/build.gradle M java/kudu-spark-tools/src/test/scala/org/apache/kudu/spark/tools/TestImportExportFiles.scala M java/kudu-spark/build.gradle M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduContextTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduRDDTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduTestSuite.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/SparkSQLTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/StreamingTest.scala M java/kudu-subprocess/build.gradle M java/kudu-test-utils/build.gradle 22 files changed, 79 insertions(+), 59 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/55/16455/3 -- To view, visit http://gerrit.cloudera.org:8080/16455 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Icc61547b6844893085de8c0f7edee50d84a248cf Gerrit-Change-Number: 16455 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16455 ) Change subject: [Java] Upgrade dependencies .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/16455/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/16455/2//COMMIT_MSG@20 PS2, Line 20: - ranger 2.0.0 -> 2.1.0 > Nice! We should also try upgrading the thirdparty build to use this. Will do. I will work on some thirdparty upgrades. http://gerrit.cloudera.org:8080/#/c/16455/2/java/kudu-spark/build.gradle File java/kudu-spark/build.gradle: http://gerrit.cloudera.org:8080/#/c/16455/2/java/kudu-spark/build.gradle@31 PS2, Line 31: > nit: extra space Done http://gerrit.cloudera.org:8080/#/c/16455/2/java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala File java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala: http://gerrit.cloudera.org:8080/#/c/16455/2/java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala@33 PS2, Line 33: import org.junit.Assert._ > nit: should we try avoiding wildcards for scala too? Same with spark.sql.fu Perhaps, but I didn't go through and rewrite code I wasn't touching. Maybe it could be a follow on. -- To view, visit http://gerrit.cloudera.org:8080/16455 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Icc61547b6844893085de8c0f7edee50d84a248cf Gerrit-Change-Number: 16455 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 15 Sep 2020 19:57:20 + Gerrit-HasComments: Yes
[kudu-CR] [Java] Upgrade dependencies
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16455 ) Change subject: [Java] Upgrade dependencies .. Patch Set 2: Code-Review+1 (3 comments) http://gerrit.cloudera.org:8080/#/c/16455/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/16455/2//COMMIT_MSG@20 PS2, Line 20: - ranger 2.0.0 -> 2.1.0 Nice! We should also try upgrading the thirdparty build to use this. http://gerrit.cloudera.org:8080/#/c/16455/2/java/kudu-spark/build.gradle File java/kudu-spark/build.gradle: http://gerrit.cloudera.org:8080/#/c/16455/2/java/kudu-spark/build.gradle@31 PS2, Line 31: nit: extra space http://gerrit.cloudera.org:8080/#/c/16455/2/java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala File java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala: http://gerrit.cloudera.org:8080/#/c/16455/2/java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala@33 PS2, Line 33: import org.junit.Assert._ nit: should we try avoiding wildcards for scala too? Same with spark.sql.functions? -- To view, visit http://gerrit.cloudera.org:8080/16455 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Icc61547b6844893085de8c0f7edee50d84a248cf Gerrit-Change-Number: 16455 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 15 Sep 2020 19:44:15 + Gerrit-HasComments: Yes
[kudu-CR] [Java] Upgrade dependencies
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16455 to look at the new patch set (#2). Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - avro 1.8.2 -> 1.10.0 - clojure 1.9.0 -> 1.10.1 - commonsIo 2.6 -> 2.8.0 - hadoop 3.2.1 -> 3.3.0 - jmh 1.23 -> 1.25.2 - log4j 2.11.2 -> 2.13.3 - mockito 3.3.3 -> 3.5.10 - protobuf 3.11.4 -> 3.13.0 - ranger 2.0.0 -> 2.1.0 - scalatest 3.0.8 -> 3.2.2 Maintenance version upgrades: - jetty 9.4.28.v20200408 -> 9.4.31.v20200723 - micrometer 1.5.1 -> 1.5.4 - netty 4.1.49.Final -> 4.1.52.Final - parquet 1.11.0 -> 1.11.1 - spark 2.4.5 -> 2.4.7 Gradle upgrades: - gradle 6.2.2 -> 6.6.1 - checkstyle 8.29 -> 8.36.1 - clojureToolsCli 0.4.1 -> 1.0.194 - spotBugs 4.0.0-beta1 -> 4.1.1 - gradle-versions-plugin 0.28.0 -> 0.33.0 - shadow 5.2.0 -> 6.0.0 - protobuf-gradle-plugin 0.8.12 -> 0.8.13 - nebula-clojure-plugin 9.4.0 -> 9.4.1 - spotbugs-gradle-plugin 4.0.2 -> 4.5.0 - gradle-errorprone-plugin 1.1.1 -> 1.2.1 - gradle-animalsniffer-plugin 1.5.0 -> 1.5.1 - gradle-scalafmt 1.7.2 -> 1.14.0 - jmh-gradle-plugin 0.5.0 -> 0.5.1 Change-Id: Icc61547b6844893085de8c0f7edee50d84a248cf --- M java/buildSrc/build.gradle M java/config/spotbugs/excludeFilter.xml M java/gradle/dependencies.gradle M java/gradle/quality.gradle M java/kudu-backup-common/build.gradle M java/kudu-backup/build.gradle M java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestKuduBackup.scala M java/kudu-client-tools/build.gradle M java/kudu-client/build.gradle M java/kudu-hive/build.gradle M java/kudu-mapreduce/build.gradle M java/kudu-spark-tools/build.gradle M java/kudu-spark-tools/src/test/scala/org/apache/kudu/spark/tools/TestImportExportFiles.scala M java/kudu-spark/build.gradle M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduContextTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduRDDTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduTestSuite.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/SparkSQLTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/StreamingTest.scala M java/kudu-subprocess/build.gradle M java/kudu-test-utils/build.gradle 22 files changed, 80 insertions(+), 57 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/55/16455/2 -- To view, visit http://gerrit.cloudera.org:8080/16455 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Icc61547b6844893085de8c0f7edee50d84a248cf Gerrit-Change-Number: 16455 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has removed a vote on this change. Change subject: [Java] Upgrade dependencies .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/16455 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: deleteVote Gerrit-Change-Id: Icc61547b6844893085de8c0f7edee50d84a248cf Gerrit-Change-Number: 16455 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16455 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/16455 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Icc61547b6844893085de8c0f7edee50d84a248cf Gerrit-Change-Number: 16455 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 15 Sep 2020 19:03:01 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16455 Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - avro 1.8.2 -> 1.10.0 - clojure 1.9.0 -> 1.10.1 - commonsIo 2.6 -> 2.8.0 - hadoop 3.2.1 -> 3.3.0 - jmh 1.23 -> 1.25.2 - log4j 2.11.2 -> 2.13.3 - mockito 3.3.3 -> 3.5.10 - protobuf 3.11.4 -> 3.13.0 - ranger 2.0.0 -> 2.1.0 - scalatest 3.0.8 -> 3.2.2 Maintenance version upgrades: - jetty 9.4.28.v20200408 -> 9.4.31.v20200723 - micrometer 1.5.1 -> 1.5.4 - netty 4.1.49.Final -> 4.1.52.Final - parquet 1.11.0 -> 1.11.1 - spark 2.4.5 -> 2.4.7 Gradle upgrades: - gradle 6.2.2 -> 6.6.1 - checkstyle 8.29 -> 8.36.1 - clojureToolsCli 0.4.1 -> 1.0.194 - spotBugs 4.0.0-beta1 -> 4.1.1 - gradle-versions-plugin 0.28.0 -> 0.33.0 - shadow 5.2.0 -> 6.0.0 - protobuf-gradle-plugin 0.8.12 -> 0.8.13 - nebula-clojure-plugin 9.4.0 -> 9.4.1 - spotbugs-gradle-plugin 4.0.2 -> 4.5.0 - gradle-errorprone-plugin 1.1.1 -> 1.2.1 - gradle-animalsniffer-plugin 1.5.0 -> 1.5.1 - gradle-scalafmt 1.7.2 -> 1.14.0 - jmh-gradle-plugin 0.5.0 -> 0.5.1 Change-Id: Icc61547b6844893085de8c0f7edee50d84a248cf --- M java/buildSrc/build.gradle M java/config/spotbugs/excludeFilter.xml M java/gradle/dependencies.gradle M java/gradle/quality.gradle M java/kudu-backup-common/build.gradle M java/kudu-backup/build.gradle M java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestKuduBackup.scala M java/kudu-client-tools/build.gradle M java/kudu-client/build.gradle M java/kudu-hive/build.gradle M java/kudu-mapreduce/build.gradle M java/kudu-spark-tools/build.gradle M java/kudu-spark-tools/src/test/scala/org/apache/kudu/spark/tools/TestImportExportFiles.scala M java/kudu-spark/build.gradle M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduContextTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduRDDTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduTestSuite.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/SparkSQLTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/StreamingTest.scala M java/kudu-subprocess/build.gradle M java/kudu-test-utils/build.gradle 22 files changed, 98 insertions(+), 80 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/55/16455/1 -- To view, visit http://gerrit.cloudera.org:8080/16455 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Icc61547b6844893085de8c0f7edee50d84a248cf Gerrit-Change-Number: 16455 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15891 ) Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependency versions. Minor version upgrades: - guava 28.2-jre -> 29.0-jre - mockito 3.2.4 -> 3.3.3 - yetus 0.11.1 -> 0.12.0 Maintenance version upgrades: - hive 3.1.1 -> 3.1.2 - httpClient 4.5.11 -> 4.5.12 - jetty 9.4.26.v20200117 -> 9.4.28.v20200408 - netty 4.1.44.Final -> 4.1.49.Final - protobuf 3.11.3 -> 3.11.4 Change-Id: I965858841502e291659d73e3e24640048b1166e9 Reviewed-on: http://gerrit.cloudera.org:8080/15891 Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong Reviewed-by: Alexey Serbin Reviewed-by: Bankim Bhavsar --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle 2 files changed, 9 insertions(+), 9 deletions(-) Approvals: Kudu Jenkins: Verified Andrew Wong: Looks good to me, approved Alexey Serbin: Looks good to me, approved Bankim Bhavsar: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/15891 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I965858841502e291659d73e3e24640048b1166e9 Gerrit-Change-Number: 15891 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Bankim Bhavsar Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/15891 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/15891 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I965858841502e291659d73e3e24640048b1166e9 Gerrit-Change-Number: 15891 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Bankim Bhavsar Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 08 May 2020 22:52:58 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15891 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/15891 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I965858841502e291659d73e3e24640048b1166e9 Gerrit-Change-Number: 15891 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 08 May 2020 22:47:45 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15891 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/15891 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I965858841502e291659d73e3e24640048b1166e9 Gerrit-Change-Number: 15891 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 08 May 2020 22:09:30 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15891 Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependency versions. Minor version upgrades: - guava 28.2-jre -> 29.0-jre - mockito 3.2.4 -> 3.3.3 - yetus 0.11.1 -> 0.12.0 Maintenance version upgrades: - hive 3.1.1 -> 3.1.2 - httpClient 4.5.11 -> 4.5.12 - jetty 9.4.26.v20200117 -> 9.4.28.v20200408 - netty 4.1.44.Final -> 4.1.49.Final - protobuf 3.11.3 -> 3.11.4 Change-Id: I965858841502e291659d73e3e24640048b1166e9 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle 2 files changed, 9 insertions(+), 9 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/91/15891/1 -- To view, visit http://gerrit.cloudera.org:8080/15891 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I965858841502e291659d73e3e24640048b1166e9 Gerrit-Change-Number: 15891 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15193 ) Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - junit 4.12 -> 4.13 - mockito 3.1.0 -> 3.2.4 - parquet 1.10.1 -> 1.11.0 - protobuf 3.10.0 -> 3.11.3 Maintenance version upgrades: - guava 28.1-jre -> 28.2-jre - hdrhistogram 2.1.11 -> 2.1.12 - httpClient 4.5.10 -> 4.5.11 - jetty 9.4.22.v20191022 -> 9.4.26.v20200117 - slf4j 1.7.29 -> 1.7.30 - spark 2.4.4 -> 2.4.5 Gradle upgrades: - checkstyle 8.26 -> 8.29 - gradle-errorprone-plugin 0.8 -> 1.1.1 - nebula-clojure-plugin 9.1.1 -> 9.4.0 - protobuf-gradle-plugin 0.8.10 -> 0.8.11 - spotbugs-gradle-plugin 2.0.1 -> 3.0.0 Change-Id: Idb6c8276364941fb294bf14d0cfc9565843ac741 Reviewed-on: http://gerrit.cloudera.org:8080/15193 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo Reviewed-by: Alexey Serbin --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle 2 files changed, 15 insertions(+), 15 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved Alexey Serbin: Looks good to me, but someone else must approve -- To view, visit http://gerrit.cloudera.org:8080/15193 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Idb6c8276364941fb294bf14d0cfc9565843ac741 Gerrit-Change-Number: 15193 Gerrit-PatchSet: 4 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Bankim Bhavsar Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Hao Hao Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15193 ) Change subject: [Java] Upgrade dependencies .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/15193 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Idb6c8276364941fb294bf14d0cfc9565843ac741 Gerrit-Change-Number: 15193 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Bankim Bhavsar Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Hao Hao Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 11 Feb 2020 05:53:19 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15193 ) Change subject: [Java] Upgrade dependencies .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/15193 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Idb6c8276364941fb294bf14d0cfc9565843ac741 Gerrit-Change-Number: 15193 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Bankim Bhavsar Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Hao Hao Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 11 Feb 2020 05:25:42 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15193 ) Change subject: [Java] Upgrade dependencies .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/15193 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Idb6c8276364941fb294bf14d0cfc9565843ac741 Gerrit-Change-Number: 15193 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Bankim Bhavsar Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Hao Hao Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 10 Feb 2020 23:48:19 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15193 ) Change subject: [Java] Upgrade dependencies .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/15193 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Idb6c8276364941fb294bf14d0cfc9565843ac741 Gerrit-Change-Number: 15193 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Bankim Bhavsar Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Hao Hao Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 10 Feb 2020 23:33:24 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15193 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: (2 comments) The primary reason is to get fixes and avoid bug/security issues. I find it's also easier to stay up-to-date with small incremental changes than large updates. http://gerrit.cloudera.org:8080/#/c/15193/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/15193/1//COMMIT_MSG@26 PS1, Line 26: checkstlye > checkstyle Done http://gerrit.cloudera.org:8080/#/c/15193/1/java/buildSrc/build.gradle File java/buildSrc/build.gradle: http://gerrit.cloudera.org:8080/#/c/15193/1/java/buildSrc/build.gradle@41 PS1, Line 41: compile "cz.alenkacz:gradle-scalafmt:1.13.0" > Missed this one in the commit message. I actually meant to remove that one for now. I saw an issue in my local build I want to understand more. -- To view, visit http://gerrit.cloudera.org:8080/15193 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Idb6c8276364941fb294bf14d0cfc9565843ac741 Gerrit-Change-Number: 15193 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Bankim Bhavsar Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Hao Hao Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 10 Feb 2020 23:02:20 + Gerrit-HasComments: Yes
[kudu-CR] [Java] Upgrade dependencies
Hello Kudu Jenkins, Adar Dembo, Hao Hao, Bankim Bhavsar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15193 to look at the new patch set (#2). Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - junit 4.12 -> 4.13 - mockito 3.1.0 -> 3.2.4 - parquet 1.10.1 -> 1.11.0 - protobuf 3.10.0 -> 3.11.3 Maintenance version upgrades: - guava 28.1-jre -> 28.2-jre - hdrhistogram 2.1.11 -> 2.1.12 - httpClient 4.5.10 -> 4.5.11 - jetty 9.4.22.v20191022 -> 9.4.26.v20200117 - slf4j 1.7.29 -> 1.7.30 - spark 2.4.4 -> 2.4.5 Gradle upgrades: - checkstyle 8.26 -> 8.29 - gradle-errorprone-plugin 0.8 -> 1.1.1 - nebula-clojure-plugin 9.1.1 -> 9.4.0 - protobuf-gradle-plugin 0.8.10 -> 0.8.11 - spotbugs-gradle-plugin 2.0.1 -> 3.0.0 Change-Id: Idb6c8276364941fb294bf14d0cfc9565843ac741 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle 2 files changed, 15 insertions(+), 15 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/93/15193/2 -- To view, visit http://gerrit.cloudera.org:8080/15193 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Idb6c8276364941fb294bf14d0cfc9565843ac741 Gerrit-Change-Number: 15193 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Bankim Bhavsar Gerrit-Reviewer: Hao Hao Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15193 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/15193/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/15193/1//COMMIT_MSG@26 PS1, Line 26: checkstlye checkstyle http://gerrit.cloudera.org:8080/#/c/15193/1/java/buildSrc/build.gradle File java/buildSrc/build.gradle: http://gerrit.cloudera.org:8080/#/c/15193/1/java/buildSrc/build.gradle@41 PS1, Line 41: compile "cz.alenkacz:gradle-scalafmt:1.13.0" Missed this one in the commit message. -- To view, visit http://gerrit.cloudera.org:8080/15193 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Idb6c8276364941fb294bf14d0cfc9565843ac741 Gerrit-Change-Number: 15193 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Bankim Bhavsar Gerrit-Reviewer: Hao Hao Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 10 Feb 2020 21:49:00 + Gerrit-HasComments: Yes
[kudu-CR] [Java] Upgrade dependencies
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/15193 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: Code-Review+1 Curious is there a reason for these upgrades or in general just good to keep them updated? -- To view, visit http://gerrit.cloudera.org:8080/15193 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Idb6c8276364941fb294bf14d0cfc9565843ac741 Gerrit-Change-Number: 15193 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Bankim Bhavsar Gerrit-Reviewer: Hao Hao Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 10 Feb 2020 21:11:54 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15193 Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - junit 4.12 -> 4.13 - mockito 3.1.0 -> 3.2.4 - parquet 1.10.1 -> 1.11.0 - protobuf 3.10.0 -> 3.11.3 Maintenance version upgrades: - guava 28.1-jre -> 28.2-jre - hdrhistogram 2.1.11 -> 2.1.12 - httpClient 4.5.10 -> 4.5.11 - jetty 9.4.22.v20191022 -> 9.4.26.v20200117 - slf4j 1.7.29 -> 1.7.30 - spark 2.4.4 -> 2.4.5 Gradle upgrades: - checkstlye 8.26 -> 8.29 - spotbugs 4.0.0-beta1 -> 4.0.0-RC1 - gradle-errorprone-plugin 0.8 -> 1.1.1 - nebula-clojure-plugin 9.1.1 -> 9.4.0 - protobuf-gradle-plugin 0.8.10 -> 0.8.11 - spotbugs-gradle-plugin 2.0.1 -> 3.0.0 Change-Id: Idb6c8276364941fb294bf14d0cfc9565843ac741 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle 2 files changed, 16 insertions(+), 16 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/93/15193/1 -- To view, visit http://gerrit.cloudera.org:8080/15193 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Idb6c8276364941fb294bf14d0cfc9565843ac741 Gerrit-Change-Number: 15193 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14746 ) Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito 2.27.0 -> 3.1.0 Minor version upgrades: - guava 27.1-jre -> 28.1-jre - hamcrest 2.1 -> 2.2 - protobuf 3.7.1 -> 3.10.0 - yetus 0.10.0 -> 0.11.1 Maintenance version upgrades: - hadoop 3.2.0 - 3.2.1 - httpClient 4.5.8 -> 4.5.10 - jetty 9.4.18.v20190429 -> 9.4.22.v20191022 - scalatest 3.0.7 -> 3.0.8 Gradle upgrades: - gradle 5.4.1 -> 5.6.4 - gradle-errorprone-plugin 0.8 -> 1.1.1 - gradle-version 0.21.0 -> 0.27.0 - gson 2.8.5 -> 2.8.6 - nebula-clojure-plugin 8.1.1 -> 9.1.1 - protobuf-gradle-plugin 0.8.8 -> 0.8.10 - shadow 5.0.0 -> 5.2.0 - spotbugs-gradle-plugin 1.7.1 -> 2.0.1 I also set checkstyle to 8.26 to ensure Gradle upgrades don’t change the version and break. Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 Reviewed-on: http://gerrit.cloudera.org:8080/14746 Tested-by: Grant Henke Reviewed-by: Alexey Serbin --- M java/buildSrc/build.gradle M java/config/checkstyle/checkstyle.xml M java/gradle/dependencies.gradle M java/gradle/quality.gradle M java/gradle/wrapper/gradle-wrapper.properties M java/gradlew 6 files changed, 43 insertions(+), 45 deletions(-) Approvals: Grant Henke: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/14746 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 Gerrit-Change-Number: 14746 Gerrit-PatchSet: 4 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14746 ) Change subject: [Java] Upgrade dependencies .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/14746 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 Gerrit-Change-Number: 14746 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 20 Nov 2019 18:53:32 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14746 ) Change subject: [Java] Upgrade dependencies .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/14746 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 Gerrit-Change-Number: 14746 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 20 Nov 2019 16:15:38 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has removed a vote on this change. Change subject: [Java] Upgrade dependencies .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/14746 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: deleteVote Gerrit-Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 Gerrit-Change-Number: 14746 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14746 ) Change subject: [Java] Upgrade dependencies .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/14746/2/java/gradle/dependencies.gradle File java/gradle/dependencies.gradle: http://gerrit.cloudera.org:8080/#/c/14746/2/java/gradle/dependencies.gradle@30 PS2, Line 30: checkstyle : "8.26", > Should this be in your commit message? Done http://gerrit.cloudera.org:8080/#/c/14746/2/java/gradle/quality.gradle File java/gradle/quality.gradle: http://gerrit.cloudera.org:8080/#/c/14746/2/java/gradle/quality.gradle@29 PS2, Line 29: toolVersion = versions.checkstyle > Got trailing whitespace here. Done -- To view, visit http://gerrit.cloudera.org:8080/14746 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 Gerrit-Change-Number: 14746 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 20 Nov 2019 15:49:18 + Gerrit-HasComments: Yes
[kudu-CR] [Java] Upgrade dependencies
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14746 to look at the new patch set (#3). Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito 2.27.0 -> 3.1.0 Minor version upgrades: - guava 27.1-jre -> 28.1-jre - hamcrest 2.1 -> 2.2 - protobuf 3.7.1 -> 3.10.0 - yetus 0.10.0 -> 0.11.1 Maintenance version upgrades: - hadoop 3.2.0 - 3.2.1 - httpClient 4.5.8 -> 4.5.10 - jetty 9.4.18.v20190429 -> 9.4.22.v20191022 - scalatest 3.0.7 -> 3.0.8 Gradle upgrades: - gradle 5.4.1 -> 5.6.4 - gradle-errorprone-plugin 0.8 -> 1.1.1 - gradle-version 0.21.0 -> 0.27.0 - gson 2.8.5 -> 2.8.6 - nebula-clojure-plugin 8.1.1 -> 9.1.1 - protobuf-gradle-plugin 0.8.8 -> 0.8.10 - shadow 5.0.0 -> 5.2.0 - spotbugs-gradle-plugin 1.7.1 -> 2.0.1 I also set checkstyle to 8.26 to ensure Gradle upgrades don’t change the version and break. Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 --- M java/buildSrc/build.gradle M java/config/checkstyle/checkstyle.xml M java/gradle/dependencies.gradle M java/gradle/quality.gradle M java/gradle/wrapper/gradle-wrapper.properties M java/gradlew 6 files changed, 43 insertions(+), 45 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/14746/3 -- To view, visit http://gerrit.cloudera.org:8080/14746 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 Gerrit-Change-Number: 14746 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14746 ) Change subject: [Java] Upgrade dependencies .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/14746/2/java/gradle/dependencies.gradle File java/gradle/dependencies.gradle: http://gerrit.cloudera.org:8080/#/c/14746/2/java/gradle/dependencies.gradle@30 PS2, Line 30: checkstyle : "8.26", Should this be in your commit message? http://gerrit.cloudera.org:8080/#/c/14746/2/java/gradle/quality.gradle File java/gradle/quality.gradle: http://gerrit.cloudera.org:8080/#/c/14746/2/java/gradle/quality.gradle@29 PS2, Line 29: toolVersion = versions.checkstyle Got trailing whitespace here. -- To view, visit http://gerrit.cloudera.org:8080/14746 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 Gerrit-Change-Number: 14746 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 20 Nov 2019 05:34:01 + Gerrit-HasComments: Yes
[kudu-CR] [Java] Upgrade dependencies
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14746 to look at the new patch set (#2). Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito 2.27.0 -> 3.1.0 Minor version upgrades: - guava 27.1-jre -> 28.1-jre - hamcrest 2.1 -> 2.2 - protobuf 3.7.1 -> 3.10.0 - yetus 0.10.0 -> 0.11.1 Maintenance version upgrades: - hadoop 3.2.0 - 3.2.1 - httpClient 4.5.8 -> 4.5.10 - jetty 9.4.18.v20190429 -> 9.4.22.v20191022 - scalatest 3.0.7 -> 3.0.8 Gradle upgrades: - gradle 5.4.1 -> 5.6.4 - gradle-errorprone-plugin 0.8 -> 1.1.1 - gradle-version 0.21.0 -> 0.27.0 - gson 2.8.5 -> 2.8.6 - nebula-clojure-plugin 8.1.1 -> 9.1.1 - protobuf-gradle-plugin 0.8.8 -> 0.8.10 - shadow 5.0.0 -> 5.2.0 - spotbugs-gradle-plugin 1.7.1 -> 2.0.1 Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 --- M java/buildSrc/build.gradle M java/config/checkstyle/checkstyle.xml M java/gradle/dependencies.gradle M java/gradle/quality.gradle M java/gradle/wrapper/gradle-wrapper.properties M java/gradlew 6 files changed, 43 insertions(+), 45 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/14746/2 -- To view, visit http://gerrit.cloudera.org:8080/14746 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 Gerrit-Change-Number: 14746 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14746 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/14746/1/java/config/checkstyle/checkstyle.xml File java/config/checkstyle/checkstyle.xml: PS1: > What was the rationale for the change here? It was a change in checkstyle on upgrade. Without it, upgrading checkstyle breaks that LineLength is not supposed to be under TreeWalker. The outcome is the same. -- To view, visit http://gerrit.cloudera.org:8080/14746 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 Gerrit-Change-Number: 14746 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 19 Nov 2019 22:24:25 + Gerrit-HasComments: Yes
[kudu-CR] [Java] Upgrade dependencies
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14746 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/14746/1/java/config/checkstyle/checkstyle.xml File java/config/checkstyle/checkstyle.xml: PS1: What was the rationale for the change here? -- To view, visit http://gerrit.cloudera.org:8080/14746 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 Gerrit-Change-Number: 14746 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 19 Nov 2019 21:41:05 + Gerrit-HasComments: Yes
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14746 Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Major version upgrades: - mockito 2.27.0 -> 3.1.0 Minor version upgrades: - guava 27.1-jre -> 28.1-jre - hamcrest 2.1 -> 2.2 - protobuf 3.7.1 -> 3.10.0 - yetus 0.10.0 -> 0.11.1 Maintenance version upgrades: - hadoop 3.2.0 - 3.2.1 - httpClient 4.5.8 -> 4.5.10 - jetty 9.4.18.v20190429 -> 9.4.22.v20191022 - scalatest 3.0.7 -> 3.0.8 Gradle upgrades: - gradle 5.4.1 -> 5.6.4 - gradle-errorprone-plugin 0.8 -> 1.1.1 - gradle-version 0.21.0 -> 0.27.0 - gson 2.8.5 -> 2.8.6 - nebula-clojure-plugin 8.1.1 -> 9.1.1 - protobuf-gradle-plugin 0.8.8 -> 0.8.10 - shadow 5.0.0 -> 5.2.0 - spotbugs-gradle-plugin 1.7.1 -> 2.0.1 Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 --- M java/buildSrc/build.gradle M java/config/checkstyle/checkstyle.xml M java/gradle/dependencies.gradle M java/gradle/wrapper/gradle-wrapper.properties M java/gradlew 5 files changed, 41 insertions(+), 45 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/14746/1 -- To view, visit http://gerrit.cloudera.org:8080/14746 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Iee6b7b178f1dd7412ecb88261ef88f282c5744e7 Gerrit-Change-Number: 14746 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13305 ) Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - mockito 2.25.1 -> 2.27.0 - yetus 0.9.0 -> 0.10.0 Maintenance version upgrades: - httpClient 4.5.7 -> 4.5.8 - jetty 9.4.15.v20190215 -> 9.4.18.v20190429 - protobuf 3.7.0 -> 3.7.1 - spark 2.4.0 -> 2.4.3 Gradle upgrades: - gradle 5.3.1 -> 5.4.1 - spotbugs-gradle-plugin 1.6.9 -> 1.7.1 - gradle-errorprone-plugin 0.0.16 -> 0.8 Change-Id: I019b1fd8706e60f9ca3aa5810a25e15c64fc85da Reviewed-on: http://gerrit.cloudera.org:8080/13305 Reviewed-by: Adar Dembo Tested-by: Kudu Jenkins --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradle/wrapper/gradle-wrapper.properties 3 files changed, 11 insertions(+), 15 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/13305 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I019b1fd8706e60f9ca3aa5810a25e15c64fc85da Gerrit-Change-Number: 13305 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13305 ) Change subject: [Java] Upgrade dependencies .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/13305 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I019b1fd8706e60f9ca3aa5810a25e15c64fc85da Gerrit-Change-Number: 13305 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 10 May 2019 17:20:16 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13305 to look at the new patch set (#2). Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - mockito 2.25.1 -> 2.27.0 - yetus 0.9.0 -> 0.10.0 Maintenance version upgrades: - httpClient 4.5.7 -> 4.5.8 - jetty 9.4.15.v20190215 -> 9.4.18.v20190429 - protobuf 3.7.0 -> 3.7.1 - spark 2.4.0 -> 2.4.3 Gradle upgrades: - gradle 5.3.1 -> 5.4.1 - spotbugs-gradle-plugin 1.6.9 -> 1.7.1 - gradle-errorprone-plugin 0.0.16 -> 0.8 Change-Id: I019b1fd8706e60f9ca3aa5810a25e15c64fc85da --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradle/wrapper/gradle-wrapper.properties 3 files changed, 11 insertions(+), 15 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/05/13305/2 -- To view, visit http://gerrit.cloudera.org:8080/13305 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I019b1fd8706e60f9ca3aa5810a25e15c64fc85da Gerrit-Change-Number: 13305 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13305 Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - mockito 2.25.1 -> 2.27.0 - yetus 0.9.0 -> 0.10.0 Maintenance version upgrades: - httpClient 4.5.7 -> 4.5.8 - jetty 9.4.15.v20190215 -> 9.4.18.v20190429 - spark 2.4.0 -> 2.4.2 Gradle upgrades: - gradle 5.3.1 -> 5.4.1 - spotbugs-gradle-plugin 1.6.9 -> 1.7.1 - gradle-errorprone-plugin 0.0.16 -> 0.8 Change-Id: I019b1fd8706e60f9ca3aa5810a25e15c64fc85da --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradle/wrapper/gradle-wrapper.properties 3 files changed, 10 insertions(+), 10 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/05/13305/1 -- To view, visit http://gerrit.cloudera.org:8080/13305 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I019b1fd8706e60f9ca3aa5810a25e15c64fc85da Gerrit-Change-Number: 13305 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12869 ) Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Some rework of the Error Prone plugin configuration was needed to get on a current version. Gradle now includes a license in the wrapper script so the custom license injection was removed. I also removed it from the properties file, since it’s configuration data rather than code and simple enough that it would count as a non-copyrightable statement of facts. Major version upgrades: - hamcrest 1.3 -> 2.1 Minor version upgrades: - guava 27.0.1-jre -> 27.1-jre - mockito 2.23.4 -> 2.25.1 - protobuf 3.6.1 -> 3.7.0 Maintenance version upgrades: - osdetector 1.6.1 -> 1.6.2 - scalatest 3.0.5 -> 3.0.7 - slf4j 1.7.25 -> 1.7.26 Gradle upgrades: - gradle 5.1.1 -> 5.3.1 - gradle-versions-plugin 0.20.0 -> 0.21.0 - shadow 4.0.4 -> 5.0.0 - nebula-clojure-plugin 8.0.0 -> 8.1.1 - spotbugs-gradle-plugin 1.6.9 -> 1.7.0 - gradle-errorprone-plugin 0.0.16 -> 0.7.1 Change-Id: I25d73cfa715c7a5ed1d572b3d3a8f63607f0fed2 Reviewed-on: http://gerrit.cloudera.org:8080/12869 Reviewed-by: Adar Dembo Tested-by: Kudu Jenkins --- M build-support/release/rat_exclude_files.txt M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradle/quality.gradle M java/gradle/wrapper.gradle M java/gradle/wrapper/gradle-wrapper.properties M java/gradlew M java/kudu-client/build.gradle 8 files changed, 47 insertions(+), 94 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/12869 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I25d73cfa715c7a5ed1d572b3d3a8f63607f0fed2 Gerrit-Change-Number: 12869 Gerrit-PatchSet: 5 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12869 ) Change subject: [Java] Upgrade dependencies .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/12869 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I25d73cfa715c7a5ed1d572b3d3a8f63607f0fed2 Gerrit-Change-Number: 12869 Gerrit-PatchSet: 4 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Thu, 28 Mar 2019 21:41:19 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12869 to look at the new patch set (#4). Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Some rework of the Error Prone plugin configuration was needed to get on a current version. Gradle now includes a license in the wrapper script so the custom license injection was removed. I also removed it from the properties file, since it’s configuration data rather than code and simple enough that it would count as a non-copyrightable statement of facts. Major version upgrades: - hamcrest 1.3 -> 2.1 Minor version upgrades: - guava 27.0.1-jre -> 27.1-jre - mockito 2.23.4 -> 2.25.1 - protobuf 3.6.1 -> 3.7.0 Maintenance version upgrades: - osdetector 1.6.1 -> 1.6.2 - scalatest 3.0.5 -> 3.0.7 - slf4j 1.7.25 -> 1.7.26 Gradle upgrades: - gradle 5.1.1 -> 5.3.1 - gradle-versions-plugin 0.20.0 -> 0.21.0 - shadow 4.0.4 -> 5.0.0 - nebula-clojure-plugin 8.0.0 -> 8.1.1 - spotbugs-gradle-plugin 1.6.9 -> 1.7.0 - gradle-errorprone-plugin 0.0.16 -> 0.7.1 Change-Id: I25d73cfa715c7a5ed1d572b3d3a8f63607f0fed2 --- M build-support/release/rat_exclude_files.txt M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradle/quality.gradle M java/gradle/wrapper.gradle M java/gradle/wrapper/gradle-wrapper.properties M java/gradlew M java/kudu-client/build.gradle 8 files changed, 47 insertions(+), 94 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/69/12869/4 -- To view, visit http://gerrit.cloudera.org:8080/12869 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I25d73cfa715c7a5ed1d572b3d3a8f63607f0fed2 Gerrit-Change-Number: 12869 Gerrit-PatchSet: 4 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12869 ) Change subject: [Java] Upgrade dependencies .. Patch Set 3: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/12869/3/java/gradle/wrapper/gradle-wrapper.properties File java/gradle/wrapper/gradle-wrapper.properties: PS3: Does this require a RAT update? -- To view, visit http://gerrit.cloudera.org:8080/12869 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I25d73cfa715c7a5ed1d572b3d3a8f63607f0fed2 Gerrit-Change-Number: 12869 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Thu, 28 Mar 2019 20:11:10 + Gerrit-HasComments: Yes
[kudu-CR] [Java] Upgrade dependencies
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12869 to look at the new patch set (#3). Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Some rework of the Error Prone plugin configuration was needed to get on a current version. Gradle now includes a license in the wrapper script so the custom license injection was removed. I also removed it from the properties file, since it’s configuration data rather than code and simple enough that it would count as a non-copyrightable statement of facts. Major version upgrades: - hamcrest 1.3 -> 2.1 Minor version upgrades: - guava 27.0.1-jre -> 27.1-jre - mockito 2.23.4 -> 2.25.1 - protobuf 3.6.1 -> 3.7.0 Maintenance version upgrades: - osdetector 1.6.1 -> 1.6.2 - scalatest 3.0.5 -> 3.0.7 - slf4j 1.7.25 -> 1.7.26 Gradle upgrades: - gradle 5.1.1 -> 5.3.1 - gradle-versions-plugin 0.20.0 -> 0.21.0 - shadow 4.0.4 -> 5.0.0 - nebula-clojure-plugin 8.0.0 -> 8.1.1 - spotbugs-gradle-plugin 1.6.9 -> 1.7.0 - gradle-errorprone-plugin 0.0.16 -> 0.7.1 Change-Id: I25d73cfa715c7a5ed1d572b3d3a8f63607f0fed2 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradle/quality.gradle M java/gradle/wrapper.gradle M java/gradle/wrapper/gradle-wrapper.properties M java/gradlew M java/kudu-client/build.gradle 7 files changed, 46 insertions(+), 94 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/69/12869/3 -- To view, visit http://gerrit.cloudera.org:8080/12869 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I25d73cfa715c7a5ed1d572b3d3a8f63607f0fed2 Gerrit-Change-Number: 12869 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12869 ) Change subject: [Java] Upgrade dependencies .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/12869 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I25d73cfa715c7a5ed1d572b3d3a8f63607f0fed2 Gerrit-Change-Number: 12869 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 27 Mar 2019 20:27:35 + Gerrit-HasComments: No
[kudu-CR] [Java] Upgrade dependencies
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12869 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/12869/1/java/gradle/dependencies.gradle File java/gradle/dependencies.gradle: http://gerrit.cloudera.org:8080/#/c/12869/1/java/gradle/dependencies.gradle@36 PS1, Line 36: errorProneJavac: "9+181-r4173-1", > The repo for it is here: https://github.com/google/error-prone-javac Ah neat, javac written in Java. OK, makes sense. -- To view, visit http://gerrit.cloudera.org:8080/12869 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I25d73cfa715c7a5ed1d572b3d3a8f63607f0fed2 Gerrit-Change-Number: 12869 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 27 Mar 2019 18:49:26 + Gerrit-HasComments: Yes
[kudu-CR] [Java] Upgrade dependencies
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12869 ) Change subject: [Java] Upgrade dependencies .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/12869/1/java/gradle/dependencies.gradle File java/gradle/dependencies.gradle: http://gerrit.cloudera.org:8080/#/c/12869/1/java/gradle/dependencies.gradle@33 PS1, Line 33: // errorprone 2.3.2+ has an issue on older JDKs see May want to clarify that this is specifically about old JDK8 versions, so that if/when we drop JDK8 support we'll know this can be upgraded. Or is this no longer an issue now that error prone is using its own javac? http://gerrit.cloudera.org:8080/#/c/12869/1/java/gradle/dependencies.gradle@36 PS1, Line 36: errorProneJavac: "9+181-r4173-1", So this is a full-fledged JDK of sorts? Is it a native dependency? -- To view, visit http://gerrit.cloudera.org:8080/12869 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I25d73cfa715c7a5ed1d572b3d3a8f63607f0fed2 Gerrit-Change-Number: 12869 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 27 Mar 2019 17:49:31 + Gerrit-HasComments: Yes
[kudu-CR] [Java] Upgrade dependencies
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12869 Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Some rework of the Error Prone plugin configuration was needed to get on a current version. Major version upgrades: - hamcrest 1.3 -> 2.1 Minor version upgrades: - guava 27.0.1-jre -> 27.1-jre - mockito 2.23.4 -> 2.25.1 - protobuf 3.6.1 -> 3.7.0 Maintenance version upgrades: - osdetector 1.6.1 -> 1.6.2 - scalatest 3.0.5 -> 3.0.7 - slf4j 1.7.25 -> 1.7.26 Gradle upgrades: - gradle 5.1.1 -> 5.3 - gradle-versions-plugin 0.20.0 -> 0.21.0 - shadow 4.0.4 -> 5.0.0 - nebula-clojure-plugin 8.0.0 -> 8.1.1 - spotbugs-gradle-plugin 1.6.9 -> 1.7.0 - gradle-errorprone-plugin 0.0.16 -> 0.7.1 Change-Id: I25d73cfa715c7a5ed1d572b3d3a8f63607f0fed2 --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradle/quality.gradle M java/gradle/wrapper/gradle-wrapper.properties M java/kudu-client/build.gradle 5 files changed, 40 insertions(+), 34 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/69/12869/1 -- To view, visit http://gerrit.cloudera.org:8080/12869 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I25d73cfa715c7a5ed1d572b3d3a8f63607f0fed2 Gerrit-Change-Number: 12869 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke
[kudu-CR] [Java] Upgrade dependencies
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12336 ) Change subject: [Java] Upgrade dependencies .. [Java] Upgrade dependencies Upgrades the Java dependencies and Gradle versions. Minor version upgrades: - flume 1.8.0 -> 1.9.0 - hadoop 3.1.1 -> 3.2.0 - yetus 0.7.0 -> 0.9.0 Maintenance version upgrades: - guava 27.0-android -> 27.0.1-android - parquet 1.10.0 -> 1.10.1 - scopt 3.7.0 -> 3.7.1 Gradle upgrades: - gradle 5.0.0 -> 5.1.1 - shadow 4.0.2 -> 4.0.4 - protobuf-gradle-plugin 0.8.7 -> 0.8.8 - nebula-clojure-plugin 7.0.1 -> 8.0.0 - spotbugs-gradle-plugin 1.6.5 -> 1.6.9 - gradle-scalafmt 1.7.0 -> 1.7.2 Change-Id: I80799a2e0a94788dfd77edde1fab6b47e5695d22 Reviewed-on: http://gerrit.cloudera.org:8080/12336 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo --- M java/buildSrc/build.gradle M java/gradle/dependencies.gradle M java/gradle/wrapper/gradle-wrapper.properties M java/gradlew 4 files changed, 14 insertions(+), 14 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/12336 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I80799a2e0a94788dfd77edde1fab6b47e5695d22 Gerrit-Change-Number: 12336 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [Java] Upgrade dependencies
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12336 ) Change subject: [Java] Upgrade dependencies .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/12336 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I80799a2e0a94788dfd77edde1fab6b47e5695d22 Gerrit-Change-Number: 12336 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 01 Feb 2019 23:01:43 + Gerrit-HasComments: No