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

(Updated Jan. 27, 2018, 12:04 a.m.)


Review request for hive, anishek, Andrew Sherman, Janaki Lahorani, Sergio Pena, 
Sahil Takiar, Thejas Nair, and Vihang Karajgaonkar.


Changes
-------

- Added extra comment to explain the fix
- Added extra test to verify that multiple notifications can be added in a 
single transaction
- Fixed comment typos


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


Repository: hive-git


Description
-------

HIVE-18526 Backport HIVE-16886 to Hive 2


Diffs (updated)
-----

  metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java 
d16d2a3cd6a08217bb5b5b7bd9226f5e19f68994 
  metastore/src/test/org/apache/hadoop/hive/metastore/TestObjectStore.java 
69e8826f53c9278a7307817c8bfa4151f36631b9 


Diff: https://reviews.apache.org/r/65330/diff/2/

Changes: https://reviews.apache.org/r/65330/diff/1-2/


Testing
-------

I wrote unit test which runs against stand-alone DB and verified that 
everything is as expected.


Thanks,

Alexander Kolbasov

Reply via email to