Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Subtask] add metalake pre event for Gravitino server #5552

Closed
Tracked by #5317
FANNG1 opened this issue Nov 12, 2024 · 2 comments · Fixed by #5583
Closed
Tracked by #5317

[Subtask] add metalake pre event for Gravitino server #5552

FANNG1 opened this issue Nov 12, 2024 · 2 comments · Fixed by #5583
Assignees
@LiuQhahah
Labels
0.8.0 Release v0.8.0 help wanted Extra attention is needed subtask Subtasks of umbrella issue

Comments

@FANNG1
Copy link
Contributor

FANNG1 commented Nov 12, 2024

Describe the subtask

  1. add corresponding MetalakePreEvent
  2. generate pre-event in MetalakeEventDispatcher
  3. add test in TestMetalakeEvent
  4. add document in gravitino-server-config.md

Please refer to the implemetation of table pre event in #5539

Parent issue

#5317
@FANNG1 FANNG1 added subtask Subtasks of umbrella issue help wanted Extra attention is needed labels Nov 12, 2024
@FANNG1 FANNG1 mentioned this issue Nov 12, 2024
Closed
7 tasks
@LiuQhahah
Copy link
Contributor

Hi @FANNG1

Please assign the task to me.

Thanks
Qiang

@FANNG1
Copy link
Contributor Author

FANNG1 commented Nov 13, 2024

@LiuQhahah , done, thanks

LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 14, 2024
@LiuQhahah
apache#5552-add-metalake-pre-event-for-Gravitino-server
7e23eab
@LiuQhahah LiuQhahah mentioned this issue Nov 14, 2024
Merged
LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 15, 2024
@LiuQhahah
Merge branch 'main' into apache#5552-add-metalake-pre-event-for-Gravi…
e78a790 
…tino-server
@FANNG1 FANNG1 added the 0.8.0 Release v0.8.0 label Nov 20, 2024
LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 20, 2024
@LiuQhahah
Merge branch 'main' into apache#5552-add-metalake-pre-event-for-Gravi…
9f2a191 
…tino-server

# Conflicts:
#	docs/gravitino-server-config.md
LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 20, 2024
@LiuQhahah
Merge remote-tracking branch 'origin/apache#5552-add-metalake-pre-eve…
fc66bb7 
…nt-for-Gravitino-server' into apache#5552-add-metalake-pre-event-for-Gravitino-server
jerryshao pushed a commit that referenced this issue Nov 21, 2024
@LiuQhahah
[#5552] feat(core): add metalake pre event to Gravitino event (#5583)
1a08d54 
### What changes were proposed in this pull request?

1. add corresponding MetalakePreEvent 
2. generate pre-event in MetalakeEventDispatcher
3. add test in TestMetalakeEvent
4. add a document in gravitino-server-config.md

### Why are the changes needed?

Fix: #5552


### Does this PR introduce _any_ user-facing change?

no

### How was this patch tested?

(Please test your changes, and provide instructions on how to test it:

UT
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@LiuQhahah LiuQhahah

Labels
0.8.0 Release v0.8.0 help wanted Extra attention is needed subtask Subtasks of umbrella issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#5552] feat(core): add metalake pre event to Gravitino event LiuQhahah/gravitino
2 participants
@FANNG1 @LiuQhahah