You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Attila Bukor (Code Review)" <ge...@cloudera.org> on 2022/01/19 11:59:23 UTC
[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 <ab...@apache.org>
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Reviewed-by: Alexey Serbin <as...@cloudera.com>
---
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
[kudu-CR] [java] Upgrade dependencies
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
Gerrit-Comment-Date: Wed, 26 Jan 2022 06:34:49 +0000
Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
Gerrit-Comment-Date: Tue, 25 Jan 2022 03:33:37 +0000
Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
[kudu-CR] [java] Upgrade dependencies
Posted by "Zoltan Chovan (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
Gerrit-Comment-Date: Thu, 20 Jan 2022 16:09:16 +0000
Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
Gerrit-Comment-Date: Tue, 25 Jan 2022 17:24:48 +0000
Gerrit-HasComments: No
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
Gerrit-Comment-Date: Mon, 24 Jan 2022 18:14:30 +0000
Gerrit-HasComments: No
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Wed, 19 Jan 2022 15:16:29 +0000
Gerrit-HasComments: No
[kudu-CR] [java] Upgrade dependencies
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
Gerrit-Comment-Date: Thu, 20 Jan 2022 16:35:56 +0000
Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
Gerrit-Comment-Date: Mon, 24 Jan 2022 16:09:05 +0000
Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
[kudu-CR] [java] Upgrade dependencies
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
Gerrit-Comment-Date: Tue, 25 Jan 2022 06:01:17 +0000
Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
Gerrit-Comment-Date: Tue, 25 Jan 2022 16:50:19 +0000
Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
Gerrit-Comment-Date: Thu, 20 Jan 2022 16:22:47 +0000
Gerrit-HasComments: Yes
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 20 Jan 2022 14:51:44 +0000
Gerrit-HasComments: No
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
[kudu-CR] [java] Upgrade dependencies
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
[kudu-CR] [java] Upgrade dependencies
Posted by "Zoltan Chovan (Code Review)" <ge...@cloudera.org>.
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 <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
Gerrit-Comment-Date: Mon, 24 Jan 2022 16:14:45 +0000
Gerrit-HasComments: No