hive-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ser...@apache.org
Subject [20/31] hive git commit: HIVE-14681: Remove AssertionError for non zero return codes in QTestUtil cleanup (Siddharth Seth reviewed by Hari Subramaniyan)
Date Mon, 12 Sep 2016 20:24:52 GMT
HIVE-14681: Remove AssertionError for non zero return codes in QTestUtil cleanup (Siddharth
Seth reviewed by Hari Subramaniyan)


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

Branch: refs/heads/hive-14535
Commit: 518789df22310b2ff95e92d0ee7cf7636daa70b3
Parents: e09f3c7
Author: Prasanth Jayachandran <prasanthj@apache.org>
Authored: Mon Sep 12 12:09:45 2016 -0700
Committer: Prasanth Jayachandran <prasanthj@apache.org>
Committed: Mon Sep 12 12:09:45 2016 -0700

----------------------------------------------------------------------
 .../util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/518789df/itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java
----------------------------------------------------------------------
diff --git a/itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java b/itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java
index 4d4a929..f456dfb 100644
--- a/itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java
+++ b/itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java
@@ -860,7 +860,8 @@ public class QTestUtil {
       SessionState.get().getConf().setBoolean("hive.test.shutdown.phase", true);
       int result = cliDriver.processLine(cleanupCommands);
       if (result != 0) {
-        Assert.fail("Failed during cleanup processLine with code=" + result);
+        LOG.error("Failed during cleanup processLine with code={}. Ignoring", result);
+        // TODO Convert this to an Assert.fail once HIVE-14682 is fixed
       }
       SessionState.get().getConf().setBoolean("hive.test.shutdown.phase", false);
     } else {


Mime
View raw message