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 <[email protected]> Authored: Mon Sep 12 12:09:45 2016 -0700 Committer: Prasanth Jayachandran <[email protected]> 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 {
