Michael Smith has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23990 )

Change subject: IMPALA-14764 (part 1): Vendor LLVM's run-clang-tidy.py
......................................................................


Patch Set 5: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/23990/5/bin/run_clang_tidy.sh
File bin/run_clang_tidy.sh:

http://gerrit.cloudera.org:8080/#/c/23990/5/bin/run_clang_tidy.sh@72
PS5, Line 72:                        -j"${CORES}" -hide-progress ${DIRS} 2> 
${TMP_STDERR} | \
nit: commit message doesn't mention adding 'hide-progress'.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia409d0bcdaefc93b81d1f3df278b2e4258122f9c
Gerrit-Change-Number: 23990
Gerrit-PatchSet: 5
Gerrit-Owner: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Comment-Date: Mon, 23 Feb 2026 22:35:58 +0000
Gerrit-HasComments: Yes

Reply via email to