hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mithun Radhakrishnan" <mithun.radhakrish...@yahoo.com>
Subject Re: Review Request: Include Table information in Hive's AddPartitionEvent
Date Wed, 07 Nov 2012 00:47:39 GMT

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


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