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 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/10615/1/bin/run_clang_tidy.sh File bin/run_clang_tidy.sh: http://gerrit.cloudera.org:8080/#/c/10615/1/bin/run_clang_tidy.sh@57 PS1, Line 57: run-clang-tidy.py -header-filter "${PIPE_DIRS%?}" -j"${CORES}" ${DIRS} 2> /dev/null > Would -quiet work? Unfortunately, -quiet only handles a few types of output. Most of the noise is from output like this: """ Fix conflicts with existing fix! The new insertion has the same insert location as an existing replacement. New replacement: /home/joe/view1/Impala/be/src/util/string-parser.h: 11416:+0:"break; " Existing replacement: /home/joe/view1/Impala/be/src/util/string-parser.h: 11416:+0:"FALLTHROUGH_INTENDED; " """ Apparently, -quiet doesn't handle that. -- 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: 1 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jim Apple <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Comment-Date: Thu, 07 Jun 2018 00:05:12 +0000 Gerrit-HasComments: Yes
