You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by mx...@apache.org on 2015/05/12 14:19:08 UTC
flink git commit: [QA] display max 100 lines of compiler warnings
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 <mx...@apache.org>
Authored: Tue May 12 13:55:58 2015 +0200
Committer: Maximilian Michels <mx...@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