Repository: flink
Updated Branches:
  refs/heads/master 57615aaa1 -> 4b53e66c4


[QA] display max 100 lines of compiler warnings

hide warning if output file does not exist


Project: http://git-wip-us.apache.org/repos/asf/flink/repo
Commit: http://git-wip-us.apache.org/repos/asf/flink/commit/4b53e66c
Tree: http://git-wip-us.apache.org/repos/asf/flink/tree/4b53e66c
Diff: http://git-wip-us.apache.org/repos/asf/flink/diff/4b53e66c

Branch: refs/heads/master
Commit: 4b53e66c447d6f91b505da52eb4cd19a65640194
Parents: 57615aa
Author: Maximilian Michels <m...@apache.org>
Authored: Tue May 12 13:55:58 2015 +0200
Committer: Maximilian Michels <m...@apache.org>
Committed: Tue May 12 14:18:46 2015 +0200

----------------------------------------------------------------------
 tools/qa-check.sh | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/flink/blob/4b53e66c/tools/qa-check.sh
----------------------------------------------------------------------
diff --git a/tools/qa-check.sh b/tools/qa-check.sh
index 6881887..ea8f9f6 100755
--- a/tools/qa-check.sh
+++ b/tools/qa-check.sh
@@ -70,7 +70,7 @@ cd flink
 export TESTS_PASSED=true
 # Store output of results in a file in the qa dir
 QA_OUTPUT="$VAR_DIR/qa_results.txt"
-rm "$QA_OUTPUT"
+rm -f "$QA_OUTPUT"
 
 append_output() {
        echo "$1"
@@ -115,7 +115,8 @@ checkCompilerWarnings() {
        if [ "$NEW_COMPILER_ERR_CNT" -gt "$OLD_COMPILER_ERR_CNT" ]; then
                append_output ":-1: The change increases the number of compiler 
warnings from $OLD_COMPILER_ERR_CNT to $NEW_COMPILER_ERR_CNT"
                append_output '```diff'
-               append_output `diff $VAR_DIR/_COMPILER_REFERENCE_WARNINGS 
$VAR_DIR/_COMPILER_NEW_WARNINGS`
+               append_output "First 100 warnings:"
+               append_output "`diff $VAR_DIR/_COMPILER_REFERENCE_WARNINGS 
$VAR_DIR/_COMPILER_NEW_WARNINGS | head -n 100`"
                append_output '```'
                TESTS_PASSED=false
        else

Reply via email to