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 partition pre event for Gravitino server #5550

Closed
Tracked by #5317
FANNG1 opened this issue Nov 12, 2024 · 1 comment · Fixed by #5590
Closed
Tracked by #5317

[Subtask] add partition pre event for Gravitino server #5550

FANNG1 opened this issue Nov 12, 2024 · 1 comment · Fixed by #5590
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 PartitionPreEvent
  2. generate pre-event in PartitionEventDispatcher
  3. add test in TestPartitionEvent
  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 the subtask Subtasks of umbrella issue label Nov 12, 2024
@FANNG1 FANNG1 mentioned this issue Nov 12, 2024
Closed
7 tasks
@FANNG1 FANNG1 added the help wanted Extra attention is needed label Nov 12, 2024
@LiuQhahah
Copy link
Contributor

Hi @FANNG1
please assign the task to me.
Thanks,
Qiang

LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 15, 2024
@LiuQhahah
apache#5550-add-partition-pre-event-for-Gravitino-server
86c28be
@LiuQhahah LiuQhahah mentioned this issue Nov 15, 2024
Merged
LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 19, 2024
@LiuQhahah
Merge branch 'main' into apache#5550-add-partition-pre-event-for-Grav…
58766a7 
…itino-server

# Conflicts:
#	docs/gravitino-server-config.md
LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 19, 2024
@LiuQhahah
apache#5550 format code style
b794ec6
@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#5550-add-partition-pre-event-for-Grav…
82ea65f 
…itino-server
LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 20, 2024
@LiuQhahah
apache#5550 update variable name and make PartitionPreEvent abstract
8394eab
LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 21, 2024
@LiuQhahah
Merge branch 'main' into apache#5550-add-partition-pre-event-for-Grav…
a15df66 
…itino-server

# Conflicts:
#	docs/gravitino-server-config.md
LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 22, 2024
@LiuQhahah
Merge branch 'main' into apache#5550-add-partition-pre-event-for-Grav…
5573d19 
…itino-server

# Conflicts:
#	core/src/test/java/org/apache/gravitino/listener/api/event/TestPartitionEvent.java
LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 22, 2024
@LiuQhahah
apache#5550 add operationType
abb2fe4
LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 22, 2024
@LiuQhahah
apache#5550 add test for operationType
f39079a
LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 22, 2024
@LiuQhahah
apache#5550 format style with spotlessApply
f07e47c
LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 24, 2024
@LiuQhahah
apache#5550 check with preEvent instead of event
6cfb3c7
LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 25, 2024
@LiuQhahah
apache#5550 the OperationStatus is UNPROCESSED instead of SUCCESS
14fe23c
LiuQhahah added a commit to LiuQhahah/gravitino that referenced this issue Nov 25, 2024
@LiuQhahah
Merge branch 'main' into apache#5550-add-partition-pre-event-for-Grav…
7382527 
…itino-server
FANNG1 pushed a commit that referenced this issue Nov 25, 2024
@LiuQhahah
[#5550] feat(core): add partition pre event to Gravitino event (#5590)
c1b9885 
### What changes were proposed in this pull request?

1. add corresponding PartitionPreEvent
2. generate pre-event in PartitionEventDispatcher
3. add test in TestPartitionEvent
4. add document in gravitino-server-config.md

### Why are the changes needed?

Fix: #5550

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

NO

### How was this patch tested?

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.

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