hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "david seraf" <...@altiscale.com>
Subject Re: Review Request 25178: HIVE-7100 Add DROP TABLE PURGE
Date Tue, 16 Sep 2014 23:52:44 GMT

-----------------------------------------------------------
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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message