Hello Alexey Serbin, Yingchun Lai, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/21613
to look at the new patch set (#10).
Change subject: [log] Support logging audit logs to a separate file
......................................................................
[log] Support logging audit logs to a separate file
Previously, audit logs were mixed with regular logs, making it
hard to isolate audit activities during troubleshooting. This
patch introduces a mechanism to log audit entries into a dedicated
log file when the flag --audit_log_record_separate is enabled on
master.
Key changes:
- Introduced a new `AuditLogSink` implementation using glog's sink
mechanism.
- Added a new macro `LOG_AUDIT()` to emit audit logs.
- Supported log rotation and optional compression for audit logs.
- Provided unit tests to verify correct log routing and file
generation.
Usage:
- `LOG(INFO)` → standard logs.
- `LOG_AUDIT(INFO)` → audit logs, routed to both audit log and
standard log if configured.
Currently considering recording the RPC requests for table
creation, deletion, alteration, and recall operations to the
audit log.
References:
- https://google.github.io/glog/stable/sinks/
Change-Id: Ie5323361befb456d91a12da7273865542f1d2430
---
M src/kudu/master/CMakeLists.txt
A src/kudu/master/audit_logger-test.cc
A src/kudu/master/audit_logger.cc
A src/kudu/master/audit_logger.h
M src/kudu/master/catalog_manager.cc
M src/kudu/master/master-test.cc
6 files changed, 517 insertions(+), 4 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/13/21613/10
--
To view, visit http://gerrit.cloudera.org:8080/21613
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie5323361befb456d91a12da7273865542f1d2430
Gerrit-Change-Number: 21613
Gerrit-PatchSet: 10
Gerrit-Owner: KeDeng <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: KeDeng <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yingchun Lai <[email protected]>
