You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by br...@apache.org on 2015/01/28 01:42:09 UTC
svn commit: r1655202 - in /hive/trunk/metastore/src:
java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
test/org/apache/hadoop/hive/metastore/TestHiveMetaStorePartitionSpecs.java
Author: brock
Date: Wed Jan 28 00:42:09 2015
New Revision: 1655202
URL: http://svn.apache.org/r1655202
Log:
HIVE-9475 - HiveMetastoreClient.tableExists does not work (Brock reviewed by Szehon)
Modified:
hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestHiveMetaStorePartitionSpecs.java
Modified: hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
URL: http://svn.apache.org/viewvc/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java?rev=1655202&r1=1655201&r2=1655202&view=diff
==============================================================================
--- hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java (original)
+++ hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java Wed Jan 28 00:42:09 2015
@@ -1209,7 +1209,7 @@ public class HiveMetaStoreClient impleme
public boolean tableExists(String databaseName, String tableName) throws MetaException,
TException, UnknownDBException {
try {
- return filterHook.filterTable(client.get_table(databaseName, tableName)) == null;
+ return filterHook.filterTable(client.get_table(databaseName, tableName)) != null;
} catch (NoSuchObjectException e) {
return false;
}
Modified: hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestHiveMetaStorePartitionSpecs.java
URL: http://svn.apache.org/viewvc/hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestHiveMetaStorePartitionSpecs.java?rev=1655202&r1=1655201&r2=1655202&view=diff
==============================================================================
--- hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestHiveMetaStorePartitionSpecs.java (original)
+++ hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestHiveMetaStorePartitionSpecs.java Wed Jan 28 00:42:09 2015
@@ -143,6 +143,8 @@ public class TestHiveMetaStorePartitionS
Table table = new Table(tableName, dbName, "", 0, 0, 0, storageDescriptor, partColumns, tableParameters, "", "", "");
hmsc.createTable(table);
+ Assert.assertTrue("Table " + dbName + "." + tableName + " does not exist",
+ hmsc.tableExists(dbName, tableName));
}