[ https://issues.apache.org/jira/browse/HIVE-2194?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13055983#comment-13055983 ]
jirapos...@reviews.apache.org commented on HIVE-2194: ----------------------------------------------------- ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/853/#review920 ----------------------------------------------------------- trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java <https://reviews.apache.org/r/853/#comment1983> Can we check for equality between origP and the old partition instead of comparing fields? trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java <https://reviews.apache.org/r/853/#comment1982> Same here trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java <https://reviews.apache.org/r/853/#comment1981> origCols.equals(oldCols)? - Paul On 2011-06-21 20:25:04, Sohan Jain wrote: bq. bq. ----------------------------------------------------------- bq. This is an automatically generated e-mail. To reply, visit: bq. https://reviews.apache.org/r/853/ bq. ----------------------------------------------------------- bq. bq. (Updated 2011-06-21 20:25:04) bq. bq. bq. Review request for hive and Paul Yang. bq. bq. bq. Summary bq. ------- bq. bq. HIVE-2038 introduced the MetaStoreEventListener abstract class that defines actions to be performed after particular events on a metastore. Improve upon that class by adding events to be performed on "alter table" and "alter partition" actions. Also, update the hive metastore to call the appropriate functions of the listeners when the events happen. bq. bq. bq. This addresses bug HIVE-2194. bq. https://issues.apache.org/jira/browse/HIVE-2194 bq. bq. bq. Diffs bq. ----- bq. bq. trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java 1138144 bq. trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java 1138144 bq. trunk/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreEventListener.java 1138144 bq. trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java 1138144 bq. trunk/metastore/src/java/org/apache/hadoop/hive/metastore/events/AlterPartitionEvent.java PRE-CREATION bq. trunk/metastore/src/java/org/apache/hadoop/hive/metastore/events/AlterTableEvent.java PRE-CREATION bq. trunk/metastore/src/test/org/apache/hadoop/hive/metastore/DummyListener.java 1138144 bq. trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java 1138144 bq. bq. Diff: https://reviews.apache.org/r/853/diff bq. bq. bq. Testing bq. ------- bq. bq. Added test cases to TestMetaStoreEventListener. bq. bq. bq. Thanks, bq. bq. Sohan bq. bq. > Add actions for alter table and alter partition events for metastore event > listeners > ------------------------------------------------------------------------------------ > > Key: HIVE-2194 > URL: https://issues.apache.org/jira/browse/HIVE-2194 > Project: Hive > Issue Type: Improvement > Components: Metastore > Reporter: Sohan Jain > Assignee: Sohan Jain > Attachments: HIVE-2194.1.patch, HIVE-2194.3.patch > > > HIVE-2038 introduced the MetaStoreEventListener abstract class that defines > actions to be performed after particular events on a metastore. Improve upon > that class by adding events to be performed on "alter table" and "alter > partition" actions. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira