You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@atlas.apache.org by Madhan Neethiraj <ma...@apache.org> on 2018/08/31 05:33:20 UTC
Review Request 68584: ATLAS-2853: updated to send
entity-notifications after successful graph transaction commit
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68584/
-----------------------------------------------------------
Review request for atlas, Apoorv Naik, Ashutosh Mestry, keval bhatt, Nixon Rodrigues, and Sarath Subramanian.
Bugs: ATLAS-2853
https://issues.apache.org/jira/browse/ATLAS-2853
Repository: atlas
Description
-------
updated to send entity notifications only after graph transaction is committed
Diffs
-----
webapp/src/main/java/org/apache/atlas/notification/EntityNotificationListenerV2.java c70011f24
Diff: https://reviews.apache.org/r/68584/diff/1/
Testing
-------
Verified that the notifications are sent post transaction commit.
Thanks,
Madhan Neethiraj
Re: Review Request 68584: ATLAS-2853: updated to send
entity-notifications after successful graph transaction commit
Posted by Madhan Neethiraj <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68584/
-----------------------------------------------------------
(Updated Aug. 31, 2018, 4:11 p.m.)
Review request for atlas, Apoorv Naik, Ashutosh Mestry, keval bhatt, Nixon Rodrigues, and Sarath Subramanian.
Changes
-------
updated to extend post-commit notificaiton sending for V1 notificaitons as well
Bugs: ATLAS-2853
https://issues.apache.org/jira/browse/ATLAS-2853
Repository: atlas
Description
-------
updated to send entity notifications only after graph transaction is committed
Diffs (updated)
-----
repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityChangeNotifier.java 783d62fb3
webapp/src/main/java/org/apache/atlas/notification/EntityNotificationListenerV2.java c70011f24
webapp/src/main/java/org/apache/atlas/notification/EntityNotificationSender.java PRE-CREATION
webapp/src/main/java/org/apache/atlas/notification/NotificationEntityChangeListener.java 1eeecefc1
Diff: https://reviews.apache.org/r/68584/diff/2/
Changes: https://reviews.apache.org/r/68584/diff/1-2/
Testing
-------
Verified that the notifications are sent post transaction commit.
Thanks,
Madhan Neethiraj