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 2015/04/27 20:30:37 UTC

hive git commit: HIVE-10490 : HBase Snapshot IF fails at run time with missing dependency of MetricsRegistry (Ashutosh Chauhan via Gunther Hagleitner)

Repository: hive
Updated Branches:
  refs/heads/master 57515d63c -> 9a16be1f2


HIVE-10490 : HBase Snapshot IF fails at run time with missing dependency of MetricsRegistry (Ashutosh Chauhan via Gunther Hagleitner)


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

Branch: refs/heads/master
Commit: 9a16be1f210cab87dd296c36cd9e9467d35f3963
Parents: 57515d6
Author: Ashutosh Chauhan <ha...@apache.org>
Authored: Mon Apr 27 11:29:16 2015 -0700
Committer: Ashutosh Chauhan <ha...@apache.org>
Committed: Mon Apr 27 11:29:16 2015 -0700

----------------------------------------------------------------------
 .../java/org/apache/hadoop/hive/hbase/HBaseStorageHandler.java | 6 ++++++
 1 file changed, 6 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/9a16be1f/hbase-handler/src/java/org/apache/hadoop/hive/hbase/HBaseStorageHandler.java
----------------------------------------------------------------------
diff --git a/hbase-handler/src/java/org/apache/hadoop/hive/hbase/HBaseStorageHandler.java b/hbase-handler/src/java/org/apache/hadoop/hive/hbase/HBaseStorageHandler.java
index 50e022d..c0eff23 100644
--- a/hbase-handler/src/java/org/apache/hadoop/hive/hbase/HBaseStorageHandler.java
+++ b/hbase-handler/src/java/org/apache/hadoop/hive/hbase/HBaseStorageHandler.java
@@ -76,6 +76,8 @@ import org.apache.hadoop.security.token.Token;
 import org.apache.hadoop.util.StringUtils;
 import org.apache.zookeeper.KeeperException;
 
+import com.yammer.metrics.core.MetricsRegistry;
+
 /**
  * HBaseStorageHandler provides a HiveStorageHandler implementation for
  * HBase.
@@ -512,6 +514,10 @@ public class HBaseStorageHandler extends DefaultStorageHandler
         TableMapReduceUtil.addDependencyJars(
           jobConf, HBaseStorageHandler.class, TableInputFormatBase.class);
       }
+      if (HiveConf.getVar(jobConf, HiveConf.ConfVars.HIVE_HBASE_SNAPSHOT_NAME) != null) {
+        // There is an extra dependency on MetricsRegistry for snapshot IF.
+        TableMapReduceUtil.addDependencyJars(jobConf, MetricsRegistry.class);
+      }
       Set<String> merged = new LinkedHashSet<String>(jobConf.getStringCollection("tmpjars"));
 
       Job copy = new Job(jobConf);