This is an automated email from the ASF dual-hosted git repository. joemcdonnell pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/impala.git
commit 9c734dd5f27c35113ad7c38806e6d1dfd83b2249 Author: Joe McDonnell <[email protected]> AuthorDate: Tue Feb 17 14:09:23 2026 -0800 IMPALA-14764 (part 2): Fix LLVM's run-clang-tidy.py stderr handling For some reason, LLVM's run-clang-tidy.py captures the stderr from Clang Tidy and outputs it to stdout. The important output about issues goes to stdout. The output to stderr is mostly garbage like "70 warnings generated". To avoid that pollution, this modifies the script to send the stderr output to stderr (which we only output if there is an actual error running clang-tidy itself). Testing: - Ran bin/run_clang_tidy.sh on Ubuntu 20 / 22 Change-Id: I6ae90fb12dc222f8fb38b8e563617216ce806f0f Reviewed-on: http://gerrit.cloudera.org:8080/23991 Reviewed-by: Joe McDonnell <[email protected]> Tested-by: Joe McDonnell <[email protected]> --- bin/llvm/run-clang-tidy.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bin/llvm/run-clang-tidy.py b/bin/llvm/run-clang-tidy.py index f4c3d0073..2cc5fdd8b 100755 --- a/bin/llvm/run-clang-tidy.py +++ b/bin/llvm/run-clang-tidy.py @@ -736,7 +736,7 @@ async def main() -> None: if result.stdout: print(result.stdout, end=("" if result.stderr else "\n")) if result.stderr: - print(result.stderr) + print(result.stderr, file=sys.stderr) except asyncio.CancelledError: if not args.hide_progress: print("\nCtrl-C detected, goodbye.")
