Joe McDonnell 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:

(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'.
Good point, added that the commit message



--
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: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Comment-Date: Tue, 24 Feb 2026 01:07:29 +0000
Gerrit-HasComments: Yes

Reply via email to