Repository: hive
Updated Branches:
  refs/heads/master b0d3cb452 -> b98fb1f1a


HIVE-18604: DropDatabase cascade fails when there is an index in the DB (Adam 
Szita, reviewed by Peter Vary)


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

Branch: refs/heads/master
Commit: b98fb1f1aa47210aa240b1c22b32525828b47df1
Parents: b0d3cb4
Author: Peter Vary <pv...@cloudera.com>
Authored: Wed Feb 14 16:05:01 2018 +0100
Committer: Peter Vary <pv...@cloudera.com>
Committed: Wed Feb 14 16:05:01 2018 +0100

----------------------------------------------------------------------
 .../hadoop/hive/metastore/HiveMetaStore.java    |  5 ++
 .../metastore/client/builder/TableBuilder.java  |  1 +
 .../client/TestAddAlterDropIndexes.java         | 74 +++++++++++---------
 .../hive/metastore/client/TestDatabases.java    |  2 +
 .../metastore/client/TestGetListIndexes.java    | 23 ++++--
 .../metastore/client/TestListPartitions.java    |  1 -
 6 files changed, 66 insertions(+), 40 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/b98fb1f1/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
----------------------------------------------------------------------
diff --git 
a/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
 
b/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
index 8dc9b6a..1f99828 100644
--- 
a/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
+++ 
b/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
@@ -4989,6 +4989,11 @@ public class HiveMetaStore extends ThriftHiveMetastore {
         long time = System.currentTimeMillis() / 1000;
         Table indexTbl = indexTable;
         if (indexTbl != null) {
+          if (!TableType.INDEX_TABLE.name().equals(indexTbl.getTableType())){
+            throw new InvalidObjectException(
+                    "The table " + indexTbl.getTableName()+ " provided as 
index table must have "
+                            + TableType.INDEX_TABLE + " table type");
+          }
           try {
             indexTbl = ms.getTable(qualified[0], qualified[1]);
           } catch (Exception e) {

http://git-wip-us.apache.org/repos/asf/hive/blob/b98fb1f1/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/client/builder/TableBuilder.java
----------------------------------------------------------------------
diff --git 
a/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/client/builder/TableBuilder.java
 
b/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/client/builder/TableBuilder.java
index 69acf3c..a5325e7 100644
--- 
a/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/client/builder/TableBuilder.java
+++ 
b/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/client/builder/TableBuilder.java
@@ -142,6 +142,7 @@ public class TableBuilder extends 
StorageDescriptorBuilder<TableBuilder> {
     dbName = index.getDbName();
     tableName = index.getIndexTableName();
     setCols(index.getSd().getCols());
+    setType(TableType.INDEX_TABLE.name());
     return this;
   }
 

http://git-wip-us.apache.org/repos/asf/hive/blob/b98fb1f1/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestAddAlterDropIndexes.java
----------------------------------------------------------------------
diff --git 
a/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestAddAlterDropIndexes.java
 
b/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestAddAlterDropIndexes.java
index 88bacbf..680a863 100644
--- 
a/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestAddAlterDropIndexes.java
+++ 
b/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestAddAlterDropIndexes.java
@@ -25,6 +25,7 @@ import java.util.stream.Collectors;
 
 import org.apache.hadoop.fs.Path;
 import org.apache.hadoop.hive.metastore.IMetaStoreClient;
+import org.apache.hadoop.hive.metastore.TableType;
 import org.apache.hadoop.hive.metastore.api.AlreadyExistsException;
 import org.apache.hadoop.hive.metastore.api.Database;
 import org.apache.hadoop.hive.metastore.api.FieldSchema;
@@ -147,7 +148,7 @@ public class TestAddAlterDropIndexes {
         .setSerdeName(INDEX_TABLE_NAME)
         .addStorageDescriptorParam(sdParamKey, sdParamValue)
         .build();
-    client.createIndex(index, buildTable(DB_NAME, INDEX_TABLE_NAME));
+    client.createIndex(index, buildIndexTable(DB_NAME, INDEX_TABLE_NAME));
 
     Index resultIndex = client.getIndex(DB_NAME, TABLE_NAME, INDEX_NAME);
     Assert.assertNotNull(resultIndex);
@@ -202,8 +203,8 @@ public class TestAddAlterDropIndexes {
     Table origTable1 = createTable(DB_NAME, origTableName1);
     Table origTable2 = createTable(DB_NAME, origTableName2);
 
-    createIndex(DB_NAME, origTable1, INDEX_NAME, buildTable(DB_NAME, 
indexNameTableName1));
-    createIndex(DB_NAME, origTable2, INDEX_NAME, buildTable(DB_NAME, 
indexNameTableName2));
+    createIndex(DB_NAME, origTable1, INDEX_NAME, buildIndexTable(DB_NAME, 
indexNameTableName1));
+    createIndex(DB_NAME, origTable2, INDEX_NAME, buildIndexTable(DB_NAME, 
indexNameTableName2));
 
     verifyIndex(DB_NAME, origTableName1, INDEX_NAME, indexNameTableName1);
     verifyIndex(DB_NAME, origTableName2, INDEX_NAME, indexNameTableName2);
@@ -217,7 +218,7 @@ public class TestAddAlterDropIndexes {
 
     // TODO: Special character should not be allowed in index name.
     Table origTable = createTable(DB_NAME, TABLE_NAME);
-    Table indexTable = buildTable(DB_NAME, INDEX_TABLE_NAME);
+    Table indexTable = buildIndexTable(DB_NAME, INDEX_TABLE_NAME);
     String indexName = "§±!;@#$%^&*()_+}{|-=[]\':|?><";
     createIndex(DB_NAME, origTable, indexName, indexTable);
     verifyIndex(DB_NAME, TABLE_NAME, indexName, INDEX_TABLE_NAME);
@@ -230,7 +231,7 @@ public class TestAddAlterDropIndexes {
     Table origTable = createTable(DB_NAME, TABLE_NAME);
     String indexName = "UPPERCASE_INDEX_NAME";
     String indexTableName = "UPPERCASE_INDEX_TABLE_NAME";
-    Table indexTable = buildTable(DB_NAME, indexTableName);
+    Table indexTable = buildIndexTable(DB_NAME, indexTableName);
     createIndex(DB_NAME, origTable, indexName, indexTable);
     verifyIndex(DB_NAME, TABLE_NAME, indexName.toLowerCase(), 
indexTableName.toLowerCase());
     client.dropIndex(DB_NAME, TABLE_NAME, indexName, true);
@@ -240,7 +241,7 @@ public class TestAddAlterDropIndexes {
   public void testCreateIndexesWithSameOrigAndIndexTable() throws Exception {
 
     Table origTable = createTable(DB_NAME, TABLE_NAME);
-    Table indexTable = buildTable(DB_NAME, INDEX_TABLE_NAME);
+    Table indexTable = buildIndexTable(DB_NAME, INDEX_TABLE_NAME);
     createIndex(DB_NAME, origTable, INDEX_NAME, indexTable);
     createIndex(DB_NAME, origTable, INDEX_NAME, indexTable);
     checkIfIndexExists(DB_NAME, TABLE_NAME, INDEX_NAME);
@@ -253,8 +254,8 @@ public class TestAddAlterDropIndexes {
     Table origTable = createTable(DB_NAME, TABLE_NAME);
     String indexTableName1 = origTable.getTableName() + "__" + INDEX_NAME + 
"_1";
     String indexTableName2 = origTable.getTableName() + "__" + INDEX_NAME + 
"_2";
-    createIndex(DB_NAME, origTable, INDEX_NAME, buildTable(DB_NAME, 
indexTableName1));
-    createIndex(DB_NAME, origTable, INDEX_NAME, buildTable(DB_NAME, 
indexTableName2));
+    createIndex(DB_NAME, origTable, INDEX_NAME, buildIndexTable(DB_NAME, 
indexTableName1));
+    createIndex(DB_NAME, origTable, INDEX_NAME, buildIndexTable(DB_NAME, 
indexTableName2));
     checkIfIndexExists(DB_NAME, TABLE_NAME, INDEX_NAME);
     client.dropIndex(DB_NAME, TABLE_NAME, INDEX_NAME, true);
   }
@@ -272,7 +273,7 @@ public class TestAddAlterDropIndexes {
   public void testCreateIndexNullIndexTableName() throws Exception {
 
     Table origTable = createTable(DB_NAME, TABLE_NAME);
-    Table indexTable = buildTable(DB_NAME, null);
+    Table indexTable = buildIndexTable(DB_NAME, null);
     createIndex(DB_NAME, origTable, INDEX_NAME, indexTable);
     checkIfIndexExists(DB_NAME, TABLE_NAME, INDEX_NAME);
   }
@@ -281,7 +282,7 @@ public class TestAddAlterDropIndexes {
   public void testCreateIndexEmptyIndexTableName() throws Exception {
 
     Table origTable = createTable(DB_NAME, TABLE_NAME);
-    Table indexTable = buildTable(DB_NAME, "");
+    Table indexTable = buildIndexTable(DB_NAME, "");
     createIndex(DB_NAME, origTable, INDEX_NAME, indexTable);
     checkIfIndexListEmpty(DB_NAME, TABLE_NAME);
   }
@@ -296,7 +297,7 @@ public class TestAddAlterDropIndexes {
     String indexTableName = origTableName1 + "__" + indexName1;
     Table origTable1 = createTable(DB_NAME, origTableName1);
     Table origTable2 = createTable(DB_NAME, origTableName2);
-    Table indexTable = buildTable(DB_NAME, indexTableName);
+    Table indexTable = buildIndexTable(DB_NAME, indexTableName);
 
     createIndex(DB_NAME, origTable1, indexName1, indexTable);
     createIndex(DB_NAME, origTable2, indexName2, indexTable);
@@ -310,7 +311,7 @@ public class TestAddAlterDropIndexes {
   public void testCreateIndexWithNonExistingDB() throws Exception {
 
     Table origTable = createTable(DB_NAME, TABLE_NAME);
-    Table indexTable = buildTable(DB_NAME, INDEX_TABLE_NAME);
+    Table indexTable = buildIndexTable(DB_NAME, INDEX_TABLE_NAME);
     createIndex("nonexistingdb", origTable, INDEX_NAME, indexTable);
     checkIfIndexListEmpty(DB_NAME, TABLE_NAME);
   }
@@ -319,14 +320,14 @@ public class TestAddAlterDropIndexes {
   public void testCreateIndexWithNullDBName() throws Exception {
 
     Table origTable = createTable(DB_NAME, TABLE_NAME);
-    Table indexTable = buildTable(DB_NAME, INDEX_TABLE_NAME);
+    Table indexTable = buildIndexTable(DB_NAME, INDEX_TABLE_NAME);
     createIndex(null, origTable, INDEX_NAME, indexTable);
   }
 
   @Test(expected = InvalidObjectException.class)
   public void testCreateIndexNonExistingOrigTable() throws Exception {
 
-    Table indexTable = buildTable(DB_NAME, INDEX_TABLE_NAME);
+    Table indexTable = buildIndexTable(DB_NAME, INDEX_TABLE_NAME);
     Index index = buildIndexWithDefaultValues();
     index.setOrigTableName("nonexistingtable");
     client.createIndex(index, indexTable);
@@ -335,7 +336,7 @@ public class TestAddAlterDropIndexes {
   @Test(expected = MetaException.class)
   public void testCreateIndexNullOrigTable() throws Exception {
 
-    Table indexTable = buildTable(DB_NAME, INDEX_TABLE_NAME);
+    Table indexTable = buildIndexTable(DB_NAME, INDEX_TABLE_NAME);
     Index index = buildIndexWithDefaultValues();
     index.setOrigTableName(null);
     client.createIndex(index, indexTable);
@@ -344,7 +345,7 @@ public class TestAddAlterDropIndexes {
   @Test
   public void testCreateIndexNullIndex() throws Exception {
     // TODO: NPE should not be thrown.
-    Table indexTable = buildTable(DB_NAME, INDEX_TABLE_NAME);
+    Table indexTable = buildIndexTable(DB_NAME, INDEX_TABLE_NAME);
     try {
       client.createIndex(null, indexTable);
       Assert.fail("TTransportException or NullPointerException should have 
happened");
@@ -372,7 +373,7 @@ public class TestAddAlterDropIndexes {
   @Test(expected = InvalidObjectException.class)
   public void testCreateIndexWithEmptyOrigTable() throws Exception {
 
-    Table indexTable = buildTable(DB_NAME, INDEX_TABLE_NAME);
+    Table indexTable = buildIndexTable(DB_NAME, INDEX_TABLE_NAME);
     Index index = buildIndexWithDefaultValues();
     index.setOrigTableName("");
     client.createIndex(index, indexTable);
@@ -384,7 +385,7 @@ public class TestAddAlterDropIndexes {
     createTable(DB_NAME, TABLE_NAME);
     Index index = buildIndexWithDefaultValues();
     index.setDbName("");
-    client.createIndex(index, buildTable(DB_NAME, INDEX_TABLE_NAME));
+    client.createIndex(index, buildIndexTable(DB_NAME, INDEX_TABLE_NAME));
   }
 
   @Test(expected = MetaException.class)
@@ -393,14 +394,14 @@ public class TestAddAlterDropIndexes {
     createTable(DB_NAME, TABLE_NAME);
     Index index = buildIndexWithDefaultValues();
     index.setIndexTableName(null);
-    client.createIndex(index, buildTable(DB_NAME, INDEX_TABLE_NAME));
+    client.createIndex(index, buildIndexTable(DB_NAME, INDEX_TABLE_NAME));
   }
 
   @Test
   public void testCreateIndexEmptyIndexName() throws Exception {
 
     Table origTable = createTable(DB_NAME, TABLE_NAME);
-    Table indexTable = buildTable(DB_NAME, INDEX_TABLE_NAME);
+    Table indexTable = buildIndexTable(DB_NAME, INDEX_TABLE_NAME);
     createIndex(DB_NAME, origTable, "", indexTable);
     checkIfIndexExists(DB_NAME, TABLE_NAME, "");
     client.dropIndex(DB_NAME, TABLE_NAME, "", true);
@@ -412,7 +413,7 @@ public class TestAddAlterDropIndexes {
     createTable(DB_NAME, TABLE_NAME);
     Index index = buildIndexWithDefaultValues();
     index.setIndexTableName("differentindextablename");
-    Table indexTable = buildTable(DB_NAME, INDEX_TABLE_NAME);
+    Table indexTable = buildIndexTable(DB_NAME, INDEX_TABLE_NAME);
     client.createIndex(index, indexTable);
     checkIfIndexListEmpty(DB_NAME, TABLE_NAME);
     List<String> tableNames = client.listTableNamesByFilter(DB_NAME, "", MAX);
@@ -429,7 +430,7 @@ public class TestAddAlterDropIndexes {
     String dbName = "second_index_db";
     createDB(dbName);
     Index index = buildIndexWithDefaultValues();
-    Table indexTable = buildTable(dbName, INDEX_TABLE_NAME);
+    Table indexTable = buildIndexTable(dbName, INDEX_TABLE_NAME);
     Assert.assertFalse(metaStore.isPathExists(new 
Path(indexTable.getSd().getLocation())));
     client.createIndex(index, indexTable);
     checkIfIndexListEmpty(DB_NAME, TABLE_NAME);
@@ -446,7 +447,7 @@ public class TestAddAlterDropIndexes {
   public void testCreateIndexDifferentColsInIndexAndIndexTable() throws 
Exception {
 
     createTable(DB_NAME, TABLE_NAME);
-    Table indexTable = buildTable(DB_NAME, INDEX_TABLE_NAME);
+    Table indexTable = buildIndexTable(DB_NAME, INDEX_TABLE_NAME);
     Index index = new IndexBuilder()
         .setDbName(DB_NAME)
         .setTableName(TABLE_NAME)
@@ -482,7 +483,7 @@ public class TestAddAlterDropIndexes {
     createTable(DB_NAME, TABLE_NAME);
     Index index = buildIndexWithDefaultValues();
     index.setSd(null);
-    client.createIndex(index, buildTable(DB_NAME, INDEX_TABLE_NAME));
+    client.createIndex(index, buildIndexTable(DB_NAME, INDEX_TABLE_NAME));
     checkIfIndexListEmpty(DB_NAME, TABLE_NAME);
   }
 
@@ -588,7 +589,7 @@ public class TestAddAlterDropIndexes {
         .setOutputFormat(oldOutputFormat)
         .addStorageDescriptorParam(oldIndexSdParamKey, oldIndexSdParamValue)
         .build();
-    client.createIndex(index, buildTable(DB_NAME, indexTableName));
+    client.createIndex(index, buildIndexTable(DB_NAME, indexTableName));
     Index oldIndex = client.getIndex(DB_NAME, TABLE_NAME, INDEX_NAME);
 
     int newCreateTime = (int) (System.currentTimeMillis() / 1000);
@@ -766,13 +767,17 @@ public class TestAddAlterDropIndexes {
   // Helper methods
 
   private Table createTable(String dbName, String tableName) throws Exception {
-    Table table = buildTable(dbName, tableName);
+    Table table = buildTable(dbName, tableName, null);
     client.createTable(table);
     return table;
   }
 
-  private Table buildTable(String dbName, String tableName) throws Exception {
-    Table table = new TableBuilder()
+  private Table buildIndexTable(String dbName, String tableName) throws 
Exception {
+    return buildTable(dbName, tableName, TableType.INDEX_TABLE);
+  }
+
+  private Table buildTable(String dbName, String tableName, TableType 
tableType) throws Exception {
+    TableBuilder tableBuilder = new TableBuilder()
         .setDbName(dbName)
         .setTableName(tableName)
         .addCol("id", "int", "test col id")
@@ -781,15 +786,20 @@ public class TestAddAlterDropIndexes {
         .setSerdeName(tableName)
         .setStoredAsSubDirectories(false)
         .addSerdeParam("testSerdeParamKey", "testSerdeParamValue")
-        .setLocation(metaStore.getWarehouseRoot() + "/" + tableName)
-        .build();
-    return table;
+        .setLocation(metaStore.getWarehouseRoot() + "/" + tableName);
+
+    if (tableType != null) {
+      tableBuilder.setType(tableType.name());
+    }
+
+    return tableBuilder.build();
   }
 
   private void createIndex(String indexName, String origTableName) throws 
Exception {
     Table origTable = createTable(DB_NAME, origTableName);
     String indexTableName = origTableName + "__" + indexName + "__";
-    createIndex(DB_NAME, origTable, indexName, buildTable(DB_NAME, 
indexTableName));
+    createIndex(DB_NAME, origTable, indexName, buildTable(DB_NAME, 
indexTableName,
+            TableType.INDEX_TABLE));
   }
 
   private void createIndex(String dbName, Table origTable, String indexName, 
Table indexTable)

http://git-wip-us.apache.org/repos/asf/hive/blob/b98fb1f1/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestDatabases.java
----------------------------------------------------------------------
diff --git 
a/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestDatabases.java
 
b/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestDatabases.java
index db80072..8af4f8b 100644
--- 
a/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestDatabases.java
+++ 
b/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestDatabases.java
@@ -20,6 +20,7 @@ package org.apache.hadoop.hive.metastore.client;
 
 import org.apache.hadoop.fs.Path;
 import org.apache.hadoop.hive.metastore.IMetaStoreClient;
+import org.apache.hadoop.hive.metastore.TableType;
 import org.apache.hadoop.hive.metastore.api.AlreadyExistsException;
 import org.apache.hadoop.hive.metastore.api.Database;
 import org.apache.hadoop.hive.metastore.api.Function;
@@ -433,6 +434,7 @@ public class TestDatabases {
     Table testIndexTable =
         new TableBuilder()
             .setDbName(databaseName)
+            .setType(TableType.INDEX_TABLE.name())
             .setTableName("test_index_table")
             .addCol("test_col", "int")
             .build();

http://git-wip-us.apache.org/repos/asf/hive/blob/b98fb1f1/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestGetListIndexes.java
----------------------------------------------------------------------
diff --git 
a/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestGetListIndexes.java
 
b/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestGetListIndexes.java
index ab3c00d..8ca40c8 100644
--- 
a/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestGetListIndexes.java
+++ 
b/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestGetListIndexes.java
@@ -22,6 +22,7 @@ import java.util.Set;
 import java.util.stream.Collectors;
 
 import org.apache.hadoop.hive.metastore.IMetaStoreClient;
+import org.apache.hadoop.hive.metastore.TableType;
 import org.apache.hadoop.hive.metastore.api.Database;
 import org.apache.hadoop.hive.metastore.api.Index;
 import org.apache.hadoop.hive.metastore.api.MetaException;
@@ -388,13 +389,17 @@ public class TestGetListIndexes {
   // Helper methods
 
   private Table createTable(String dbName, String tableName) throws Exception {
-    Table table = buildTable(dbName, tableName);
+    Table table = buildTable(dbName, tableName, null);
     client.createTable(table);
     return table;
   }
 
-  private Table buildTable(String dbName, String tableName) throws Exception {
-    Table table = new TableBuilder()
+  private Table buildIndexTable(String dbName, String tableName) throws 
Exception {
+    return buildTable(dbName, tableName, TableType.INDEX_TABLE);
+  }
+
+  private Table buildTable(String dbName, String tableName, TableType 
tableType) throws Exception {
+    TableBuilder tableBuilder = new TableBuilder()
         .setDbName(dbName)
         .setTableName(tableName)
         .addCol("id", "int", "test col id")
@@ -403,9 +408,13 @@ public class TestGetListIndexes {
         .setSerdeName(tableName)
         .setStoredAsSubDirectories(false)
         .addSerdeParam("testSerdeParamKey", "testSerdeParamValue")
-        .setLocation(metaStore.getWarehouseRoot() + "/" + tableName)
-        .build();
-    return table;
+        .setLocation(metaStore.getWarehouseRoot() + "/" + tableName);
+
+    if (tableType != null){
+      tableBuilder.setType(tableType.name());
+    }
+
+    return tableBuilder.build();
   }
 
   private Index createIndex(Table origTable, String indexName) throws 
Exception {
@@ -414,7 +423,7 @@ public class TestGetListIndexes {
     String origTableName = origTable.getTableName();
     String indexTableName = origTableName + "__" + indexName + "__";
     Index index = buildIndex(dbName, origTableName, indexName, indexTableName);
-    client.createIndex(index, buildTable(dbName, indexTableName));
+    client.createIndex(index, buildTable(dbName, indexTableName, 
TableType.INDEX_TABLE));
     return client.getIndex(dbName, origTableName, indexName);
   }
 

http://git-wip-us.apache.org/repos/asf/hive/blob/b98fb1f1/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestListPartitions.java
----------------------------------------------------------------------
diff --git 
a/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestListPartitions.java
 
b/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestListPartitions.java
index fdabbe4..e3b1198 100644
--- 
a/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestListPartitions.java
+++ 
b/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestListPartitions.java
@@ -57,7 +57,6 @@ import static junit.framework.TestCase.assertNotNull;
 import static junit.framework.TestCase.assertNull;
 import static junit.framework.TestCase.assertTrue;
 import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.fail;
 
 /**

Reply via email to