----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26257/ -----------------------------------------------------------
Review request for hive and Thejas Nair. Bugs: HIVE-3781 https://issues.apache.org/jira/browse/HIVE-3781 Repository: hive-git Description ------- An event listener must be called for any DDL activity. For example, create_index, drop_index today does not call metaevent listener. Diffs ----- common/src/java/org/apache/hadoop/hive/conf/HiveConf.java b0cb98b itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java 3b5f65f itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java f5e35b8 metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java 47eca29 metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreEventListener.java ec1dca2 metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java 637a39a metastore/src/java/org/apache/hadoop/hive/metastore/events/AddIndexEvent.java PRE-CREATION metastore/src/java/org/apache/hadoop/hive/metastore/events/AlterIndexEvent.java PRE-CREATION metastore/src/java/org/apache/hadoop/hive/metastore/events/DropIndexEvent.java PRE-CREATION metastore/src/java/org/apache/hadoop/hive/metastore/events/PreAddIndexEvent.java PRE-CREATION metastore/src/java/org/apache/hadoop/hive/metastore/events/PreAlterIndexEvent.java PRE-CREATION metastore/src/java/org/apache/hadoop/hive/metastore/events/PreDropIndexEvent.java PRE-CREATION metastore/src/java/org/apache/hadoop/hive/metastore/events/PreEventContext.java dbc3247 metastore/src/test/org/apache/hadoop/hive/metastore/DummyListener.java 37daf52 metastore/src/test/org/apache/hadoop/hive/metastore/DummyPreListener.java 9a943b2 ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java d5374bc ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 0f7af9a ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 4e58ad8 ql/src/java/org/apache/hadoop/hive/ql/plan/DropIndexDesc.java 5ef14ef ql/src/test/org/apache/hadoop/hive/ql/metadata/TestHive.java 153908c ql/src/test/queries/clientpositive/drop_index.q 54ef823 ql/src/test/results/clientnegative/alter_concatenate_indexed_table.q.out 19fbf51 ql/src/test/results/clientpositive/alter_concatenate_indexed_table.q.out ffcbcf9 ql/src/test/results/clientpositive/index_auth.q.out 385b639 ql/src/test/results/clientpositive/index_auto.q.out d1f5fa2 ql/src/test/results/clientpositive/index_auto_file_format.q.out eca656f ql/src/test/results/clientpositive/index_auto_mult_tables.q.out ab0f06f ql/src/test/results/clientpositive/index_auto_mult_tables_compact.q.out 20fa238 ql/src/test/results/clientpositive/index_auto_multiple.q.out 2408d48 ql/src/test/results/clientpositive/index_auto_partitioned.q.out da57bea ql/src/test/results/clientpositive/index_auto_self_join.q.out 94ded26 ql/src/test/results/clientpositive/index_auto_update.q.out 85f26ac ql/src/test/results/clientpositive/index_bitmap_auto_partitioned.q.out 473df0a ql/src/test/results/clientpositive/index_bitmap_compression.q.out b5a172f ql/src/test/results/clientpositive/index_compression.q.out 241c2e9 ql/src/test/results/clientpositive/index_serde.q.out 5b4cbfc ql/src/test/results/clientpositive/ql_rewrite_gbtoidx.q.out f7da06c ql/src/test/results/clientpositive/show_indexes_edge_cases.q.out cc6a405 ql/src/test/results/clientpositive/show_indexes_syntax.q.out bc96359 ql/src/test/results/clientpositive/union_view.q.out bbbf5e7 Diff: https://reviews.apache.org/r/26257/diff/ Testing ------- Thanks, Navis Ryu