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

[Improvement] make IcebergTableOperationDispatcher interface more extendible #5370

Open
FANNG1 opened this issue Oct 30, 2024 · 0 comments · May be fixed by #5369
Open

[Improvement] make IcebergTableOperationDispatcher interface more extendible #5370

FANNG1 opened this issue Oct 30, 2024 · 0 comments · May be fixed by #5369
Assignees
@FANNG1
Labels
improvement Improvements on everything
Milestone
Gravitino 0.7.0

Comments

@FANNG1
Copy link
Contributor

FANNG1 commented Oct 30, 2024

What would you like to be improved?

Some customer may need to do some custom logic in IcebergTableOperationDispatcher, needs more information like HTTP request.

How should we improve?

Add IcebergRESTContext to IcebergTableOperationDispatcher, which contains the catalog name and HTTP request information.
@FANNG1 FANNG1 added the improvement Improvements on everything label Oct 30, 2024
@FANNG1 FANNG1 self-assigned this Oct 30, 2024
@FANNG1 FANNG1 added this to the Gravitino 0.7.0 milestone Oct 30, 2024
@FANNG1 FANNG1 linked a pull request Oct 30, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@FANNG1 FANNG1

Labels
improvement Improvements on everything
Projects
None yet
Milestone
Gravitino 0.7.0
1 participant
@FANNG1