search

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.09k stars 343 forks source link

[#5551] feat(core): add topic pre-event to Gravitino event. #5614

Closed LindaSummer closed 21 hours ago

LindaSummer commented 2 days ago

What changes were proposed in this pull request?

Add pre-events for topic events, including AlterTopicPreEvent, CreateTopicPreEvent, DropTopicPreEvent, ListTopicPreEvent and LoadTopicPreEvent.

Why are the changes needed?

Fix: #5551

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Related Unit Tests have been added.

FANNG1 commented 1 day ago

@LindaSummer , could you add LICENSE header to xxEvent to fix rat check error?

jerryshao commented 1 day ago

Can you please fix the license header issue for your newly added files?

LindaSummer commented 1 day ago

Hi @FANNG1 , @jerryshao ,

Of course, I will add license headers today.😄

Best Regards, Edward

xunliu commented 1 day ago

hi @LindaSummer Thank you for your contribution! I was trying to contact you, This is my email liuxun@apache.org, Please send an email to me, Thank you :-)

LindaSummer commented 1 day ago

hi @LindaSummer Thank you for your contribution! I was trying to contact you, This is my email liuxun@apache.org, Please send an email to me, Thank you :-)

Hi @xunliu ,

I'm very happy to have the chance to get in touch. 😊 I have sent a greeting email to you.

Best Regards, Edward

FANNG1 commented 21 hours ago

@LindaSummer , merged to main, thanks for your contribution!