search

datastrato / gravitino

World's most powerful data catalog service with providing a high-performance, geo-distributed and federated metadata lake.
https://datastrato.ai/docs/
Apache License 2.0
347 stars 150 forks source link

[#2999][#2997] improvement(partition): add partition operation dispatcher #3221

Closed mchades closed 1 week ago

mchades commented 2 weeks ago

What changes were proposed in this pull request?

Why are the changes needed?

Fix: #2997 #2999

Does this PR introduce any user-facing change?

no

How was this patch tested?

tests added

mchades commented 1 week ago

@jerryshao All comments are resolved, do you want to take another look?

jerryshao commented 1 week ago

Yeah, I will.

jerryshao commented 1 week ago

How do you verify that the partition operation is running in the isolated classloader, is it possible to add the test case to cover this?

mchades commented 1 week ago

How do you verify that the partition operation is running in the isolated classloader, is it possible to add the test case to cover this?

@jerryshao test for isolated classloader added, please help review again