You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tez.apache.org by ss...@apache.org on 2015/05/29 20:42:39 UTC

tez git commit: TEZ-2503. findbugs version isn't reported properly in test-patch report. (sseth)

Repository: tez
Updated Branches:
  refs/heads/master 42b7756eb -> fafa804c9


TEZ-2503. findbugs version isn't reported properly in test-patch report. (sseth)


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

Branch: refs/heads/master
Commit: fafa804c953aa528c839be7a14d0ccbdd35c75f9
Parents: 42b7756
Author: Siddharth Seth <ss...@apache.org>
Authored: Fri May 29 11:42:23 2015 -0700
Committer: Siddharth Seth <ss...@apache.org>
Committed: Fri May 29 11:42:23 2015 -0700

----------------------------------------------------------------------
 CHANGES.txt               | 1 +
 build-tools/test-patch.sh | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tez/blob/fafa804c/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index 7f240c1..6fb6051 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -8,6 +8,7 @@ INCOMPATIBLE CHANGES
   TEZ-2468. Change the minimum Java version to Java 7.
 
 ALL CHANGES:
+  TEZ-2503. findbugs version isn't reported properly in test-patch report.
   TEZ-2198. Fix sorter spill counts.
   TEZ-1883. Change findbugs version to 3.x.
   TEZ-2440. Sorter should check for indexCacheList.size() in flush().

http://git-wip-us.apache.org/repos/asf/tez/blob/fafa804c/build-tools/test-patch.sh
----------------------------------------------------------------------
diff --git a/build-tools/test-patch.sh b/build-tools/test-patch.sh
index 0ac2bac..d41c54b 100755
--- a/build-tools/test-patch.sh
+++ b/build-tools/test-patch.sh
@@ -525,7 +525,7 @@ buildAndInstall () {
 ###############################################################################
 ### Check there are no changes in the number of Findbugs warnings
 checkFindbugsWarnings () {
-  findbugs_version=$(${AWK} 'match($0, /findbugs-maven-plugin:[^:]*:findbugs/) { print substr($0, RSTART + 22, RLENGTH - 31); exit }' "${PATCH_DIR}/patchFindBugsOutput${module_suffix}.txt")
+  findbugs_version=$(${AWK} 'match($0, /findbugs-maven-plugin:[^:]*:findbugs/) { print substr($0, RSTART + 22, RLENGTH - 31); exit }' "${PATCH_DIR}/patchFindBugsOutput.txt")
   echo ""
   echo ""
   echo "======================================================================"