-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25178/
-----------------------------------------------------------
(Updated Sept. 16, 2014, 11:52 p.m.)
Review request for hive and Xuefu Zhang.
Changes
-------
patch 10 fixes the last remaining review comments
Bugs: HIVE-7100
https://issues.apache.org/jira/browse/HIVE-7100
Repository: hive-git
Description
-------
Add PURGE option to DROP TABLE command to skip saving table data to the trash
Diffs (updated)
-----
hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatPartitionPublish.java
c2a0f5f
hcatalog/webhcat/svr/src/test/java/org/apache/hive/hcatalog/templeton/tool/TestTempletonUtils.java
af952f2
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
33745e4
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
8765d53
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreFsImpl.java
cff0718
metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java
e86a90a
metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreFS.java a141793
metastore/src/java/org/apache/hadoop/hive/metastore/Warehouse.java a32507d
ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 56bcf1c
ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java f40f5f7
ql/src/java/org/apache/hadoop/hive/ql/metadata/SessionHiveMetaStoreClient.java
6d18884
ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 05cde3e
ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 25cd3a5
ql/src/java/org/apache/hadoop/hive/ql/plan/DropTableDesc.java ba30e1f
ql/src/test/org/apache/hadoop/hive/ql/metadata/TestHive.java 406aae9
ql/src/test/org/apache/hadoop/hive/ql/metadata/TestHiveRemote.java 1a5ba87
ql/src/test/queries/clientpositive/drop_table_purge.q PRE-CREATION
ql/src/test/results/clientpositive/drop_table_purge.q.out PRE-CREATION
Diff: https://reviews.apache.org/r/25178/diff/
Testing
-------
added code test and added QL test. Tests passed in CI, but other, unrelated
tests failed.
Thanks,
david seraf
