Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Hive >> mail # dev >> Review Request: Include Table information in Hive's AddPartitionEvent


Copy link to this message
-
Re: Review Request: Include Table information in Hive's AddPartitionEvent

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

(Updated Nov. 7, 2012, 12:47 a.m.)
Review request for hive, Ashutosh Chauhan and Francis Liu.
Description
-------

This has to do with a minor overhaul of the HCatalog notifications that we're attempting in HCATALOG-546.
It is proposed that HCatalog's notifications (on Add/Drop of Partitions) provide details to identify the affected partitions.
Using the Partition object in AddPartitionEvent, one is able to retrieve the values of the partition-keys and the name of the Table. However, the partition-keys themselves aren't available (since the Table instance isn't part of the AddPartitionEvent).
Adding the table-reference to the AddPartitionEvent and DropPartitionEvent classes will expose all the info we need. (The alternative is to query the metastore for the table's schema and use the partition-keys from there. :/)
Diffs
-----

  http://svn.apache.org/repos/asf/hive/branches/branch-0.9/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java 1405924
  http://svn.apache.org/repos/asf/hive/branches/branch-0.9/metastore/src/java/org/apache/hadoop/hive/metastore/events/AddPartitionEvent.java 1405924
  http://svn.apache.org/repos/asf/hive/branches/branch-0.9/metastore/src/java/org/apache/hadoop/hive/metastore/events/DropPartitionEvent.java 1405924

Diff: https://reviews.apache.org/r/7910/diff/
Testing
-------
Thanks,

Mithun Radhakrishnan