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.

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

[Kernel] Shadejackson into Kernel-Api for internal JSON operations #3587

Merged
merged 2 commits into from
Aug 26, 2024

Conversation

raveeram-db
Copy link
Collaborator

@raveeram-db raveeram-db commented Aug 21, 2024

Description

Adds Jackson as a general purpose JSON parsing library into the Kernel-Api jar. The library is shaded to avoid version dependency conflicts downstream for connector developers.

How was this patch tested?

Ran existing tests.

@vkorukanti vkorukanti changed the title [Kernel][WIP] Shade Jackson into Kernel-Api for use as a general JSON… [Kernel] Shadejackson into Kernel-Api for internal JSON operations Aug 26, 2024
@raveeram-db raveeram-db force-pushed the AddShadedJacksonDep branch 2 times, most recently from 1ab1d23 to be2279a Compare August 26, 2024 04:47
@vkorukanti vkorukanti merged commit 5c6b451 into delta-io:master Aug 26, 2024
13 of 16 checks passed
longvu-db pushed a commit to longvu-db/delta that referenced this pull request Aug 28, 2024
…elta-io#3587)

## Description
Adds Jackson as a general-purpose JSON parsing library into the
Kernel-Api jar. The library is shaded to avoid version dependency
conflicts downstream for connector developers.

## How was this patch tested?
Ran existing tests.
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.

2 participants