Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/10615 )
Change subject: IMPALA-7132: Filter out useless output from run_clang_tidy.sh ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/10615/2/bin/run_clang_tidy.sh File bin/run_clang_tidy.sh: http://gerrit.cloudera.org:8080/#/c/10615/2/bin/run_clang_tidy.sh@57 PS2, Line 57: run-clang-tidy.py -quiet -header-filter "${PIPE_DIRS%?}" -j"${CORES}" ${DIRS} 2> /dev/null > What if something catastrophically bad happens with the script that is orth Good idea, changed this to write stderr to a tempfile and only output it on error. I tested it by adding an invalid import to run-clang-tidy.py and verifying that it outputs the error. -- To view, visit http://gerrit.cloudera.org:8080/10615 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I63c46a7d57295eba38fac8ab49c7a15d2802df1d Gerrit-Change-Number: 10615 Gerrit-PatchSet: 2 Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Jim Apple <jbapple-imp...@apache.org> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Comment-Date: Mon, 16 Jul 2018 23:09:14 +0000 Gerrit-HasComments: Yes