Repository navigation
[Improvement] Support event dispatching for catalog connection tests #12566
Description
What would you like to be improved?
CatalogEventDispatcher currently delegates both pre-create and existing-catalog connection tests without emitting pre, success, or failure events. This prevents listeners and auditing integrations from observing these operations.
How should we improve?
Add event dispatching for all three testConnection overloads and cover their success and failure paths with unit tests. Follow the existing catalog event conventions for operation context and exceptions. Reuse the existing audit formatting and redaction behavior.
Activity
ss666 commentedon Aug 22, 2026
Hi @mchades. Could you assign it to me?
zhang-arvin commentedon Aug 23, 2026
I would like to work on this issue.
[apache#12566] feat(core): Add event dispatching for catalog connecti…
mchades commentedon Aug 26, 2026
@ss666 You are welcome to submit a PR directly, and I will review it promptly.
ss666 commentedon Aug 26, 2026
Many thanks for your reply, @mchades. Since I haven’t started working on this yet, and both zhang-arvin and VedantMadane have already contributed great PRs, I think it might be better to wait for the community to review their PRs first, so we can avoid duplicating the work by submitting another PR. I’ve already left some comments on #12571.
fix: feat: dispatch events for catalog connection tests
[apache#12566] feat(core): Add event dispatching for catalog connecti…
[apache#12566] feat(core): Add event dispatching for catalog connecti…
[apache#12566] feat(core): Add event dispatching for catalog connecti…
[apache#12566] feat(core): Address review comments for catalog connec…
singhmrinal02 commentedon Sep 16, 2026
Hi, I'd like to work on this issue as a first contribution to Apache Gravitino. Could you please assign it to me?
Zhou-Kan commentedon Sep 23, 2026
@mchades could I work on this ticket as my first step here? Thanks
[apache#12566] fix(core): Protect connection test events and complete…
[apache#12566] fix(core): Align connection test events with catalog c…
fix: feat: dispatch events for catalog connection tests
[apache#12566] fix(core): Protect connection test events and complete…
[apache#12566] fix(core): Align connection test events with catalog c…
[apache#12566] fix(server): Return 403 for connection test vetoes
codewithsinchana commentedon Oct 26, 2026
Can you please assign it to me , I would like to work on it
mchades commentedon Oct 26, 2026
sorry @codewithsinchana , #12585 almost done
[apache#12566] feat(core): Dispatch events for catalog connection tes…