-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/75053/
-----------------------------------------------------------
(Updated June 24, 2024, 3:10 p.m.)
Review request for atlas, Ashutosh Mestry, chaitali, Disha Talreja, Jayendra
Parab, Pinal Shah, Radhika Kundam, Sarath Subramanian, Sheetal Shah, and
Sidharth Mishra.
Changes
-------
updated to address the review comment
Bugs: ATLAS-4881
https://issues.apache.org/jira/browse/ATLAS-4881
Repository: atlas
Description
-------
- reduced noise in Atlas server log file by changing log levels during
notification processing from info to debug level
- error handling during notificaiton processing updated to not retry if failure
was due to invalid data (like entity-type not found)
- updated metrics log to include the total time taken to process a notification
Diffs (updated)
-----
common/src/main/java/org/apache/atlas/utils/AtlasPerfMetrics.java c72b2c3e2
webapp/src/main/java/org/apache/atlas/notification/NotificationHookConsumer.java
7b02ac449
webapp/src/main/java/org/apache/atlas/notification/preprocessor/HiveDbDDLPreprocessor.java
dcff0939d
webapp/src/main/java/org/apache/atlas/notification/preprocessor/HivePreprocessor.java
083e343b0
webapp/src/main/java/org/apache/atlas/notification/preprocessor/HiveTableDDLPreprocessor.java
83d4d7c1a
webapp/src/main/java/org/apache/atlas/notification/preprocessor/PreprocessorContext.java
f930d9f35
Diff: https://reviews.apache.org/r/75053/diff/2/
Changes: https://reviews.apache.org/r/75053/diff/1-2/
Testing (updated)
-------
- verified that notification preprocessing doesn't print logs in info level
- pre-commit tests run:
https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/1661/
Thanks,
Madhan Neethiraj
