You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Navis Ryu <na...@nexr.com> on 2014/10/02 03:44:49 UTC

Review Request 26257: Index related events should be delivered to metastore event listener

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