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: Code-Review+2 (1 comment) bootstrap_system.sh changes make sense to me. If we don't need the new toolchain, it would be nice for that to be removed. I'm going ahead with +2 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 When this is rebased on top of https://gerrit.cloudera.org/#/c/16843/1 , do you still need to go to 69-7644f7fe9c, or can this use 68-7644f7fe9c? -- 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 19:36:04 +0000 Gerrit-HasComments: Yes
