You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by ha...@apache.org on 2017/02/17 22:16:02 UTC

hive git commit: HIVE-15969 : Failures in TestRemoteHiveMetaStore, TestSetUGIOnOnlyServer (Slim Bouguerra via Thejas Nair)

Repository: hive
Updated Branches:
  refs/heads/master 5c293716a -> e618bd1b4


HIVE-15969 : Failures in TestRemoteHiveMetaStore, TestSetUGIOnOnlyServer (Slim Bouguerra via Thejas Nair)

Signed-off-by: Ashutosh Chauhan <ha...@apache.org>


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

Branch: refs/heads/master
Commit: e618bd1b45bc45f96b27698fd0c97fee6b9c57e6
Parents: 5c29371
Author: Slim Bouguerra <sl...@gmail.com>
Authored: Fri Feb 17 14:15:15 2017 -0800
Committer: Ashutosh Chauhan <ha...@apache.org>
Committed: Fri Feb 17 14:15:15 2017 -0800

----------------------------------------------------------------------
 .../apache/hadoop/hive/metastore/HiveMetaStoreClient.java    | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/e618bd1b/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
----------------------------------------------------------------------
diff --git a/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java b/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
index 70f3a6b..7002620 100644
--- a/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
+++ b/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
@@ -739,12 +739,14 @@ public class HiveMetaStoreClient implements IMetaStoreClient {
         hook.commitCreateTable(tbl);
       }
       success = true;
-    } catch (Exception e){
-      LOG.error("Got exception from createTable", e);
     }
     finally {
       if (!success && (hook != null)) {
-        hook.rollbackCreateTable(tbl);
+        try {
+          hook.rollbackCreateTable(tbl);
+        } catch (Exception e){
+          LOG.error("Create rollback failed with", e);
+        }
       }
     }
   }