-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48723/
-----------------------------------------------------------
(Updated June 16, 2016, 12:21 p.m.)
Review request for atlas.
Changes
-------
Addressed review comments from Madhan.
Bugs: ATLAS-901
https://issues.apache.org/jira/browse/ATLAS-901
Repository: atlas
Description
-------
The description of the approach is documented on JIRA here:
https://issues.apache.org/jira/browse/ATLAS-901?focusedCommentId=15331480&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15331480
Diffs (updated)
-----
docs/src/site/twiki/Configuration.twiki 0e122fe
notification/src/main/java/org/apache/atlas/hook/AtlasHook.java 71029b0
notification/src/main/java/org/apache/atlas/hook/FailedMessagesLogger.java
PRE-CREATION
notification/src/main/java/org/apache/atlas/kafka/KafkaNotification.java
1ee62d2
notification/src/main/java/org/apache/atlas/notification/NotificationException.java
d9d89df
notification/src/test/java/org/apache/atlas/hook/AtlasHookTest.java 16cb0f0
notification/src/test/java/org/apache/atlas/kafka/KafkaNotificationTest.java
219bd70
Diff: https://reviews.apache.org/r/48723/diff/
Testing
-------
Tested by integrating with hive hook. Brought down Kafka when the hook has to
send messages. Ensured that the message is logged into the log file.
Added some UTs for the changes.
Thanks,
Hemanth Yamijala
