Laszlo Gaal (Cloudera) has posted comments on this change. ( http://gerrit.cloudera.org:8080/18437 )
Change subject: IMPALA-11259: Bump Kudu version to 1.16 release (5cd6779) ...................................................................... Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/18437/2/buildall.sh File buildall.sh: http://gerrit.cloudera.org:8080/#/c/18437/2/buildall.sh@339 PS2, Line 339: 5cd6779 Since we are picking up a specific released version, I'd prefer if we could use the release tag 1.16.0 instead of a raw Git hash. While this is just syntactic sugar (tag 1.16.0 points at exactly this git hash), it would still help traceability a bit. I checked that `git fetch origin 1.16.0` would run and fetch exactly this git hash from github.com/apache/kudu. -- To view, visit http://gerrit.cloudera.org:8080/18437 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I044e24d7402080447188355780a5f1c846a87103 Gerrit-Change-Number: 18437 Gerrit-PatchSet: 3 Gerrit-Owner: Tamas Mate <tma...@apache.org> Gerrit-Reviewer: Laszlo Gaal (Cloudera) <laszlo.g...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Tamas Mate <tma...@apache.org> Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com> Gerrit-Comment-Date: Thu, 21 Apr 2022 12:50:55 +0000 Gerrit-HasComments: Yes