You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by th...@apache.org on 2015/01/26 20:03:47 UTC
svn commit: r1654871 - in
/hive/branches/branch-1.0/metastore/src/java/org/apache/hadoop/hive/metastore:
HiveMetaStoreClient.java IMetaStoreClient.java
Author: thejas
Date: Mon Jan 26 19:03:47 2015
New Revision: 1654871
URL: http://svn.apache.org/r1654871
Log:
HIVE-3280 : Make HiveMetaStoreClient a public API (Thejas Nair, reviewed by Alan Gates)
Modified:
hive/branches/branch-1.0/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
hive/branches/branch-1.0/metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java
Modified: hive/branches/branch-1.0/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
URL: http://svn.apache.org/viewvc/hive/branches/branch-1.0/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java?rev=1654871&r1=1654870&r2=1654871&view=diff
==============================================================================
--- hive/branches/branch-1.0/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java (original)
+++ hive/branches/branch-1.0/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java Mon Jan 26 19:03:47 2015
@@ -46,6 +46,8 @@ import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.apache.hadoop.hive.common.ObjectPair;
import org.apache.hadoop.hive.common.ValidTxnList;
+import org.apache.hadoop.hive.common.classification.InterfaceAudience.Public;
+import org.apache.hadoop.hive.common.classification.InterfaceStability.Unstable;
import org.apache.hadoop.hive.conf.HiveConf;
import org.apache.hadoop.hive.conf.HiveConf.ConfVars;
import org.apache.hadoop.hive.conf.HiveConfUtil;
@@ -137,7 +139,11 @@ import org.apache.thrift.transport.TTran
/**
* Hive Metastore Client.
+ * The public implementation of IMetaStoreClient. Methods not inherited from IMetaStoreClient
+ * are not public and can change. Hence this is marked as unstable.
*/
+@Public
+@Unstable
public class HiveMetaStoreClient implements IMetaStoreClient {
ThriftHiveMetastore.Iface client = null;
private TTransport transport = null;
Modified: hive/branches/branch-1.0/metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java
URL: http://svn.apache.org/viewvc/hive/branches/branch-1.0/metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java?rev=1654871&r1=1654870&r2=1654871&view=diff
==============================================================================
--- hive/branches/branch-1.0/metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java (original)
+++ hive/branches/branch-1.0/metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java Mon Jan 26 19:03:47 2015
@@ -40,6 +40,8 @@ import java.util.List;
import java.util.Map;
import org.apache.hadoop.hive.common.ObjectPair;
+import org.apache.hadoop.hive.common.classification.InterfaceAudience.Public;
+import org.apache.hadoop.hive.common.classification.InterfaceStability.Evolving;
import org.apache.hadoop.hive.metastore.api.AggrStats;
import org.apache.hadoop.hive.metastore.api.AlreadyExistsException;
import org.apache.hadoop.hive.metastore.api.ColumnStatistics;
@@ -74,9 +76,10 @@ import org.apache.hadoop.hive.metastore.
import org.apache.hadoop.hive.metastore.api.UnknownTableException;
/**
- * TODO Unnecessary when the server sides for both dbstore and filestore are
- * merged
+ * Wrapper around hive metastore thrift api
*/
+@Public
+@Evolving
public interface IMetaStoreClient {
/**