You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by sz...@apache.org on 2015/10/14 03:58:49 UTC

hive git commit: HIVE-12168 : Addendum to HIVE-12038 (Szehon, reviewed by Sergey)

Repository: hive
Updated Branches:
  refs/heads/master 1d2e5eed3 -> 072665b13


HIVE-12168 : Addendum to HIVE-12038 (Szehon, reviewed by Sergey)


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

Branch: refs/heads/master
Commit: 072665b1320d839d8b6690228288ad770ba92f45
Parents: 1d2e5ee
Author: Szehon Ho <sz...@cloudera.com>
Authored: Tue Oct 13 18:58:29 2015 -0700
Committer: Szehon Ho <sz...@cloudera.com>
Committed: Tue Oct 13 18:58:29 2015 -0700

----------------------------------------------------------------------
 .../src/main/java/org/apache/hive/ptest/execution/JIRAService.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/072665b1/testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java
----------------------------------------------------------------------
diff --git a/testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java b/testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java
index 03a6321..ff1dbd4 100644
--- a/testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java
+++ b/testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/JIRAService.java
@@ -118,7 +118,7 @@ class JIRAService {
       comments.add(mPatch);
     }
     comments.add("");
-    if (error) {
+    if (error && numTestsExecuted == 0) {
       comments.add(formatError("-1 due to build exiting with an error"));
     } else {
       if (addedTests.size() > 0) {