apache / gravitino

World's most powerful open data catalog for building a high-performance, geo-distributed and federated metadata lake.
https://gravitino.apache.org
Apache License 2.0
1.1k stars 348 forks source link

[Improvement] integrate audit log framework for Iceberg REST server #5556

Closed FANNG1 closed 1 week ago

FANNG1 commented 2 weeks ago

What would you like to be improved?

3973 add an audit log framework and provides a default audit writer, but it couldn't work with IcebergRESTServer, the main reason is the Iceberg REST event is defined in iceberg-rest-server module, while the default writer defined in core module.

How should we improve?

To make the default audit works for both Gravitino server and Iceberg REST server, I propose to add an auditInfo interface, all events should implement it, the default audit log writer could get the audit info from all the event.

FANNG1 commented 2 weeks ago

welcome comments, cc @hanwxx @jerryshao