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 <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Thu, 20 Jan 2022 16:22:47 +0000 Gerrit-HasComments: Yes
