gravitino icon indicating copy to clipboard operation
gravitino copied to clipboard

[#9022] improve(core): Refactor to add the managed table operations

Open jerryshao opened this issue 1 month ago • 3 comments

What changes were proposed in this pull request?

This PR adds a new managed table operations class for managed table CRUD in Gravitino.

Why are the changes needed?

This is the prerequisite for refactoring the generic lakehouse catalog.

Fix: #9022

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Add UTs to cover the code.

jerryshao avatar Nov 11 '25 18:11 jerryshao

@jerryshao I have used the branch branch-lance-namespace-dev to replace the old one branch-lance-namepspace-dev, please just raise a PR based on it.

yuqi1129 avatar Nov 13 '25 03:11 yuqi1129

one minor comment. https://github.com/apache/gravitino/pull/9073#discussion_r2521330979

mchades avatar Nov 14 '25 02:11 mchades

lgtm. Do you have more comments? @yuqi1129

I have no more comments, let's wait https://github.com/apache/gravitino/pull/9073#issuecomment-3530548242 to be resolved.

yuqi1129 avatar Nov 14 '25 11:11 yuqi1129