jerryshao commented on code in PR #12891:
URL: https://github.com/apache/gravitino/pull/12891#discussion_r3932092345


##########
server-common/src/main/java/org/apache/gravitino/server/web/HttpAuditFilter.java:
##########
@@ -183,6 +192,19 @@ public void doFilter(ServletRequest request, 
ServletResponse response, FilterCha
                     eventSource);
             eventBus.get().dispatchEvent(event);
           }
+        } else if (!RequestContext.isOperationFailureFired()

Review Comment:
   Went with the docs route rather than a config switch — added a note under 
"Audit Logging" in `docs/gravitino-server-config.md` calling out the Iceberg 
REST `/v1/config`-style volume concern and pointing at log rotation/retention 
config. Happy to add a config switch instead if you'd still prefer one. Commit 
4afae98.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to