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 2014/11/24 20:49:57 UTC
svn commit: r1641456 - in /hive/branches/branch-0.14:
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
ql/src/test/queries/clientpositive/database_drop.q
ql/src/test/results/clientpositive/database_drop.q.out
Author: hashutosh
Date: Mon Nov 24 19:49:56 2014
New Revision: 1641456
URL: http://svn.apache.org/r1641456
Log:
HIVE-5664 : Drop cascade database fails when the db has any tables with indexes (Venki Korukanti via Ashutosh Chauhan)
Modified:
hive/branches/branch-0.14/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
hive/branches/branch-0.14/ql/src/test/queries/clientpositive/database_drop.q
hive/branches/branch-0.14/ql/src/test/results/clientpositive/database_drop.q.out
Modified: hive/branches/branch-0.14/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
URL: http://svn.apache.org/viewvc/hive/branches/branch-0.14/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java?rev=1641456&r1=1641455&r2=1641456&view=diff
==============================================================================
--- hive/branches/branch-0.14/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java (original)
+++ hive/branches/branch-0.14/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java Mon Nov 24 19:49:56 2014
@@ -667,7 +667,7 @@ public class HiveMetaStoreClient impleme
for (String table : tableList) {
try {
// Subclasses can override this step (for example, for temporary tables)
- dropTable(name, table, deleteData, false);
+ dropTable(name, table, deleteData, true);
} catch (UnsupportedOperationException e) {
// Ignore Index tables, those will be dropped with parent tables
}
Modified: hive/branches/branch-0.14/ql/src/test/queries/clientpositive/database_drop.q
URL: http://svn.apache.org/viewvc/hive/branches/branch-0.14/ql/src/test/queries/clientpositive/database_drop.q?rev=1641456&r1=1641455&r2=1641456&view=diff
==============================================================================
--- hive/branches/branch-0.14/ql/src/test/queries/clientpositive/database_drop.q (original)
+++ hive/branches/branch-0.14/ql/src/test/queries/clientpositive/database_drop.q Mon Nov 24 19:49:56 2014
@@ -75,6 +75,12 @@ CREATE EXTERNAL TABLE extab1(id INT, nam
STORED AS TEXTFILE
LOCATION 'file:${system:test.tmp.dir}/dbcascade/extab1';
+-- add a table, create index (give a name for index table)
+CREATE TABLE temp_tbl3 (id INT, name STRING);
+LOAD DATA LOCAL INPATH '../../data/files/kv1.txt' into table temp_tbl3;
+CREATE INDEX temp_tbl3_idx ON TABLE temp_tbl3(id) AS 'COMPACT' with DEFERRED REBUILD IN TABLE temp_tbl3_idx_tbl;
+ALTER INDEX temp_tbl3_idx ON temp_tbl3 REBUILD;
+
-- drop the database with cascade
DROP DATABASE db5 CASCADE;
Modified: hive/branches/branch-0.14/ql/src/test/results/clientpositive/database_drop.q.out
URL: http://svn.apache.org/viewvc/hive/branches/branch-0.14/ql/src/test/results/clientpositive/database_drop.q.out?rev=1641456&r1=1641455&r2=1641456&view=diff
==============================================================================
--- hive/branches/branch-0.14/ql/src/test/results/clientpositive/database_drop.q.out (original)
+++ hive/branches/branch-0.14/ql/src/test/results/clientpositive/database_drop.q.out Mon Nov 24 19:49:56 2014
@@ -352,6 +352,42 @@ POSTHOOK: type: CREATETABLE
#### A masked pattern was here ####
POSTHOOK: Output: database:db5
POSTHOOK: Output: db5@extab1
+PREHOOK: query: -- add a table, create index (give a name for index table)
+CREATE TABLE temp_tbl3 (id INT, name STRING)
+PREHOOK: type: CREATETABLE
+PREHOOK: Output: database:db5
+PREHOOK: Output: db5@temp_tbl3
+POSTHOOK: query: -- add a table, create index (give a name for index table)
+CREATE TABLE temp_tbl3 (id INT, name STRING)
+POSTHOOK: type: CREATETABLE
+POSTHOOK: Output: database:db5
+POSTHOOK: Output: db5@temp_tbl3
+PREHOOK: query: LOAD DATA LOCAL INPATH '../../data/files/kv1.txt' into table temp_tbl3
+PREHOOK: type: LOAD
+#### A masked pattern was here ####
+PREHOOK: Output: db5@temp_tbl3
+POSTHOOK: query: LOAD DATA LOCAL INPATH '../../data/files/kv1.txt' into table temp_tbl3
+POSTHOOK: type: LOAD
+#### A masked pattern was here ####
+POSTHOOK: Output: db5@temp_tbl3
+PREHOOK: query: CREATE INDEX temp_tbl3_idx ON TABLE temp_tbl3(id) AS 'COMPACT' with DEFERRED REBUILD IN TABLE temp_tbl3_idx_tbl
+PREHOOK: type: CREATEINDEX
+PREHOOK: Input: db5@temp_tbl3
+POSTHOOK: query: CREATE INDEX temp_tbl3_idx ON TABLE temp_tbl3(id) AS 'COMPACT' with DEFERRED REBUILD IN TABLE temp_tbl3_idx_tbl
+POSTHOOK: type: CREATEINDEX
+POSTHOOK: Input: db5@temp_tbl3
+POSTHOOK: Output: db5@temp_tbl3_idx_tbl
+PREHOOK: query: ALTER INDEX temp_tbl3_idx ON temp_tbl3 REBUILD
+PREHOOK: type: ALTERINDEX_REBUILD
+PREHOOK: Input: db5@temp_tbl3
+PREHOOK: Output: db5@temp_tbl3_idx_tbl
+POSTHOOK: query: ALTER INDEX temp_tbl3_idx ON temp_tbl3 REBUILD
+POSTHOOK: type: ALTERINDEX_REBUILD
+POSTHOOK: Input: db5@temp_tbl3
+POSTHOOK: Output: db5@temp_tbl3_idx_tbl
+POSTHOOK: Lineage: temp_tbl3_idx_tbl._bucketname SIMPLE [(temp_tbl3)temp_tbl3.FieldSchema(name:INPUT__FILE__NAME, type:string, comment:), ]
+POSTHOOK: Lineage: temp_tbl3_idx_tbl._offsets EXPRESSION [(temp_tbl3)temp_tbl3.FieldSchema(name:BLOCK__OFFSET__INSIDE__FILE, type:bigint, comment:), ]
+POSTHOOK: Lineage: temp_tbl3_idx_tbl.id SIMPLE [(temp_tbl3)temp_tbl3.FieldSchema(name:id, type:int, comment:null), ]
PREHOOK: query: -- drop the database with cascade
DROP DATABASE db5 CASCADE
PREHOOK: type: DROPDATABASE
@@ -369,6 +405,8 @@ PREHOOK: Output: db5@part_tab3
PREHOOK: Output: db5@temp_tbl
PREHOOK: Output: db5@temp_tbl2
PREHOOK: Output: db5@temp_tbl2_view
+PREHOOK: Output: db5@temp_tbl3
+PREHOOK: Output: db5@temp_tbl3_idx_tbl
PREHOOK: Output: db5@temp_tbl_view
POSTHOOK: query: -- drop the database with cascade
DROP DATABASE db5 CASCADE
@@ -387,5 +425,7 @@ POSTHOOK: Output: db5@part_tab3
POSTHOOK: Output: db5@temp_tbl
POSTHOOK: Output: db5@temp_tbl2
POSTHOOK: Output: db5@temp_tbl2_view
+POSTHOOK: Output: db5@temp_tbl3
+POSTHOOK: Output: db5@temp_tbl3_idx_tbl
POSTHOOK: Output: db5@temp_tbl_view
#### A masked pattern was here ####