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

Reply via email to