flink-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From m..@apache.org
Subject flink git commit: [QA] display max 100 lines of compiler warnings
Date Tue, 12 May 2015 12:19:08 GMT
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 <mxm@apache.org>
Authored: Tue May 12 13:55:58 2015 +0200
Committer: Maximilian Michels <mxm@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


Mime
View raw message