Skip to content

[Kernel] [CatalogManaged] [UC] Simple UCCatalogManagedClient #4780

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.

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

Merged

Conversation

scottsand-db
Copy link
Collaborator

@scottsand-db scottsand-db commented Jun 17, 2025

🥞 Stacked PR

Use this link to review incremental changes.


Which Delta project/connector is this regarding?

  • Spark
  • Standalone
  • Flink
  • Kernel
  • Other (fill in here)

Description

Create a new jar delta-unity and add a UCCatalogManagedClient inside of it, along with a simple test.

More advanced tests will come soon (in future PRs).

How was this patch tested?

New simple unit test.

Does this PR introduce any user-facing changes?

No.

@scottsand-db scottsand-db changed the title get a simple UCCatalogManagedClient working e2e + test [Kernel] [CatalogManaged] [UC] Simple UCCatalogManagedClient Jun 17, 2025
@scottsand-db scottsand-db self-assigned this Jun 17, 2025
@scottsand-db scottsand-db force-pushed the stack/uc_kernel_catalog_managed_1b branch from 239d9a8 to 582b7ba Compare June 18, 2025 21:09
@scottsand-db scottsand-db force-pushed the stack/uc_kernel_catalog_managed_1b branch from 28a01d7 to 50b6668 Compare June 23, 2025 17:56
@scottsand-db scottsand-db requested a review from raveeram-db June 23, 2025 17:57
Copy link
Collaborator

@raveeram-db raveeram-db left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Collaborator

@nicklan nicklan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice, looks great. left a few comments

@scottsand-db scottsand-db force-pushed the stack/uc_kernel_catalog_managed_1b branch from 18747e4 to 95c361c Compare June 25, 2025 00:22
Copy link
Collaborator

@nicklan nicklan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks!

@scottsand-db scottsand-db merged commit a87a338 into delta-io:master Jun 25, 2025
20 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants