Re: Review Request 65985: HIVE-18783: ALTER TABLE post-commit listener does not include the transactional listener responses

[Sergio Pena via Review Board]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65985/
-----------------------------------------------------------

(Updated April 4, 2018, 4:06 p.m.)


Review request for hive, Alexander Kolbasov, Sahil Takiar, and Vihang 
Karajgaonkar.


Bugs: HIVE-18783
    https://issues.apache.org/jira/browse/HIVE-18783


Repository: hive-git


Description
-------

HIVE-16164 introduced a mechanism to pass HMS notification events ID to the 
post-commit listeners for all DDL operations, but it didn't add it to the ALTER 
TABLE event. This patch in review adds the same behavior for ALTER TABLE events.


Diffs (updated)
-----

  
itests/hcatalog-unit/src/test/java/org/apache/hive/hcatalog/listener/TestDbNotificationListener.java
 5459554fec911b253583df1f528e5abf134f055b 
  
standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java
 ed1b8c5cc2a4cb974dd37419c6b5f0601a035232 
  
standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
 8539fea42fa2743381833ab3137579caeac64672 
  
standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/IHMSHandler.java
 f59f40bc33367cff7a8d0d24d0e200b16c2c30e5 


Diff: https://reviews.apache.org/r/65985/diff/3/

Changes: https://reviews.apache.org/r/65985/diff/2-3/


Testing
-------

All tests passed.


Thanks,

Sergio Pena