Browse Source

HADOOP-8584. test-patch.sh should not immediately exit when no tests are added or modified. Contributed by Colin Patrick McCabe

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/trunk@1359902 13f79535-47bb-0310-9956-ffa450edef68
Eli Collins 13 years ago
parent
commit
9a2c19e582
2 changed files with 6 additions and 2 deletions
  1. 3 2
      dev-support/test-patch.sh
  2. 3 0
      hadoop-common-project/hadoop-common/CHANGES.txt

+ 3 - 2
dev-support/test-patch.sh

@@ -921,8 +921,9 @@ fi
 checkTests
 checkTests
 (( RESULT = RESULT + $? ))
 (( RESULT = RESULT + $? ))
 applyPatch
 applyPatch
-(( RESULT = RESULT + $? ))
-if [[ $RESULT != 0 ]] ; then
+APPLY_PATCH_RET=$?
+(( RESULT = RESULT + $APPLY_PATCH_RET ))
+if [[ $APPLY_PATCH_RET != 0 ]] ; then
   submitJiraComment 1
   submitJiraComment 1
   cleanupAndExit 1
   cleanupAndExit 1
 fi
 fi

+ 3 - 0
hadoop-common-project/hadoop-common/CHANGES.txt

@@ -320,6 +320,9 @@ Branch-2 ( Unreleased changes )
     HADOOP-8566. AvroReflectSerializer.accept(Class) throws a NPE if the class has no 
     HADOOP-8566. AvroReflectSerializer.accept(Class) throws a NPE if the class has no 
     package (primitive types and arrays). (tucu)
     package (primitive types and arrays). (tucu)
 
 
+    HADOOP-8584. test-patch.sh should not immediately exit when no
+    tests are added or modified. (Colin Patrick McCabe via eli)
+
   BREAKDOWN OF HDFS-3042 SUBTASKS
   BREAKDOWN OF HDFS-3042 SUBTASKS
 
 
     HADOOP-8220. ZKFailoverController doesn't handle failure to become active
     HADOOP-8220. ZKFailoverController doesn't handle failure to become active