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: Add DROP TABLE PURGE
Date Tue, 09 Sep 2014 18:51:30 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25178/
-----------------------------------------------------------

(Updated Sept. 9, 2014, 6:51 p.m.)


Review request for hive and Xuefu Zhang.


Changes
-------

latest patch from HIVE-7100 includes documentation updates and responses to RB comments


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
be7134f 
  hcatalog/webhcat/svr/src/test/java/org/apache/hive/hcatalog/templeton/tool/TestTempletonUtils.java
af952f2 
  itests/hive-unit/src/test/java/org/apache/hive/jdbc/miniHS2/TestHiveServer2.java da51a55

  metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java 9489949 
  metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java a94a7a3 
  metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreFsImpl.java cff0718 
  metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java cbdba30 
  metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreFS.java a141793 
  metastore/src/java/org/apache/hadoop/hive/metastore/Warehouse.java 613b709 
  ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java cd017d8 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java e387b8f 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/SessionHiveMetaStoreClient.java 4cf98d8 
  ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java f31a409 
  ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 32db0c7 
  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