Joe McDonnell has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16844 )

Change subject: IMPALA-10385: Fix RPM repo ID capitalization for Centos 8.3
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/16844/1/bin/impala-config.sh
File bin/impala-config.sh:

http://gerrit.cloudera.org:8080/#/c/16844/1/bin/impala-config.sh@71
PS1, Line 71: 69-7644f7fe9c
> On a related note: toolchain build #68-xxxx was a mistake: I forgot to publ
https://gerrit.cloudera.org/#/c/16843/ was merged with 68, so we can't delete 
it.

https://github.com/apache/impala/blob/master/bin/impala-config.sh#L71

So, now I get it, this change intentionally goes to 69-7644f7fe9c, and that is 
fine.



--
To view, visit http://gerrit.cloudera.org:8080/16844
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I224beb1189ce25ae66ecd78d70757537e117805a
Gerrit-Change-Number: 16844
Gerrit-PatchSet: 1
Gerrit-Owner: Laszlo Gaal <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Laszlo Gaal <[email protected]>
Gerrit-Comment-Date: Wed, 09 Dec 2020 20:15:29 +0000
Gerrit-HasComments: Yes

Reply via email to