sachinnn99 opened a new pull request, #10694: URL: https://github.com/apache/gravitino/pull/10694
### What changes were proposed in this pull request? Fix the events documentation in `docs/gravitino-server-config.md` to match the actual event classes in the codebase: - Add missing partition post-events and topic pre-events (entire rows were absent) - Add missing disable/enable events for catalog, metalake, and policy operations - Add missing fileset events (`GetFileLocationEvent`, `ListFilesEvent` + failure variants) - Add missing model events (`GetModelVersionUriEvent`, `ListModelVersionInfosEvent`, `ListModelVersionPreEvent` + variants) - Add missing role event (`OverridePrivilegesEvent` + failure/pre variants) - Add missing owner failure events (`SetOwnerFailureEvent`, `GetOwnerFailureEvent`) - Fix wrong casing (`FileSet` → `Fileset`) for all fileset events - Fix typo (`ListTagInfoFailureEvent` → `ListTagsInfoFailureEvent`) - Fix incorrect policy event names (`ListAssociatedPoliciesForMetadataObject*` → correct names like `ListPolicyInfos*`, `GetPolicyForMetadataObject*`, etc.) - Fix duplicate `PurgeTableFailureEvent` (was listed in place of success event `PurgeTableEvent`) - Fix duplicate `AlterModelVersionPreEvent` in model pre-events ### Why are the changes needed? The events documentation was out of sync with the codebase — 76 events missing, 16 wrong/nonexistent events documented. Fix: #10690 ### Does this PR introduce _any_ user-facing change? No. Documentation-only change. ### How was this patch tested? Compared all 438 concrete event class names across `core/` and `iceberg/` modules against the documented event names — verified 0 missing, 0 extra, and 0 duplicates after the fix. -- 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]
