Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/20142 )
Change subject: IMPALA-12212: Bump Maven to 3.9.2, pull dependencies in parallel ...................................................................... Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/20142/2/bin/impala-config.sh File bin/impala-config.sh: http://gerrit.cloudera.org:8080/#/c/20142/2/bin/impala-config.sh@505 PS2, Line 505: export IMPALA_CXX_COMPILER=${IMPALA_CXX_COMPILER-default} Could we use the long-form name here? I had no idea what -B was. -- To view, visit http://gerrit.cloudera.org:8080/20142 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I75d05215effc724f5bd471646fb352f37443e185 Gerrit-Change-Number: 20142 Gerrit-PatchSet: 3 Gerrit-Owner: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Comment-Date: Fri, 30 Jun 2023 22:41:15 +0000 Gerrit-HasComments: Yes
