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