You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by se...@apache.org on 2016/11/01 22:20:10 UTC
[04/11] hive git commit: HIVE-15081:
RetryingMetaStoreClient.getProxy(HiveConf,
Boolean) doesn't match constructor of HiveMetaStoreClient (Rui reviewed by
Sergey)
HIVE-15081: RetryingMetaStoreClient.getProxy(HiveConf, Boolean) doesn't match constructor of HiveMetaStoreClient (Rui 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/a7cf6549
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/a7cf6549
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/a7cf6549
Branch: refs/heads/hive-14535
Commit: a7cf6549f136efd65531f859c1e08cce26885b37
Parents: 0e446dd
Author: Rui Li <li...@apache.org>
Authored: Mon Oct 31 15:29:24 2016 +0800
Committer: Rui Li <sh...@cn.ibm.com>
Committed: Mon Oct 31 15:29:24 2016 +0800
----------------------------------------------------------------------
.../java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java | 2 +-
.../apache/hadoop/hive/metastore/RetryingMetaStoreClient.java | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hive/blob/a7cf6549/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
----------------------------------------------------------------------
diff --git a/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java b/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
index 41385f7..9a4c61a 100644
--- a/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
+++ b/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
@@ -1647,7 +1647,7 @@ public class MetaStoreUtils {
}
for (int i = 0; i < parameterTypes.length; i++) {
Class<?> clazz = parameterTypes[i];
- if (!(clazz.isInstance(initargs[i]))) {
+ if (initargs[i] != null && !(clazz.isInstance(initargs[i]))) {
throw new IllegalArgumentException("Object : " + initargs[i]
+ " is not an instance of " + clazz);
}
http://git-wip-us.apache.org/repos/asf/hive/blob/a7cf6549/metastore/src/java/org/apache/hadoop/hive/metastore/RetryingMetaStoreClient.java
----------------------------------------------------------------------
diff --git a/metastore/src/java/org/apache/hadoop/hive/metastore/RetryingMetaStoreClient.java b/metastore/src/java/org/apache/hadoop/hive/metastore/RetryingMetaStoreClient.java
index f0bd438..96d8248 100644
--- a/metastore/src/java/org/apache/hadoop/hive/metastore/RetryingMetaStoreClient.java
+++ b/metastore/src/java/org/apache/hadoop/hive/metastore/RetryingMetaStoreClient.java
@@ -84,8 +84,8 @@ public class RetryingMetaStoreClient implements InvocationHandler {
public static IMetaStoreClient getProxy(
HiveConf hiveConf, boolean allowEmbedded) throws MetaException {
- return getProxy(hiveConf, new Class[]{HiveConf.class, Boolean.class},
- new Object[]{hiveConf, allowEmbedded}, null, HiveMetaStoreClient.class.getName()
+ return getProxy(hiveConf, new Class[]{HiveConf.class, HiveMetaHookLoader.class, Boolean.class},
+ new Object[]{hiveConf, null, allowEmbedded}, null, HiveMetaStoreClient.class.getName()
);
}