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

[KYUUBI #6101] Bump Jackson from 2.15.0 to 2.15.4 #6110

Closed
wants to merge 2 commits into from
Closed

[KYUUBI #6101] Bump Jackson from 2.15.0 to 2.15.4 #6110

wants to merge 2 commits into from

Conversation

Deepali1211
Copy link
Contributor

🔍 Description

Issue References 🔗

This pull request fixes #6101

Describe Your Solution 🔧

This updates the jackson version from 2.15.0 to 2.15.4

Types of changes 🔖

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪

Behavior Without This Pull Request ⚰️

Behavior With This Pull Request 🎉

Related Unit Tests

Checklist 📝

Be nice. Be informative.

@pan3793
Copy link
Member

pan3793 commented Feb 28, 2024 •
edited
Loading

seems there are issues on performing shading with multi-version artifact, need more investigation

@Deepali1211
Copy link
Contributor Author

@pan3793 All tests have passed now.

@Deepali1211
Copy link
Contributor Author

[Hive Test (8, -Dhive.archive.mirror=https://archive.apache.org/dist/hive/hive-2.3.9 -Dhive.archiv...](https://github.com/apache/kyuubi/actions/runs/8083005789/job/22085080370?pr=6110#logs)
This check has failed after running for 6 hrs, could be intermittent.
Do you have option to restart this particular check?

@pan3793 pan3793 added this to the v1.9.0 milestone Feb 29, 2024
@pan3793 pan3793 closed this in 8026e72 Feb 29, 2024
@pan3793
Copy link
Member

pan3793 commented Feb 29, 2024

The test failure is irrelevant, thanks, merged to master.

@Deepali1211 Deepali1211 deleted the upgrade-jackson branch February 29, 2024 06:57
@pan3793
Copy link
Member

pan3793 commented Feb 29, 2024

@Deepali1211 you may want to link your email [email protected] to your GitHub account, otherwise your contribution won't be counted by GitHub.

@Deepali1211
Copy link
Contributor Author

@pan3793 Thank you for the suggestion. I have linked my personal email Id. Do you mean it has to be org email if they want to count my contributions.

@pan3793
Copy link
Member

pan3793 commented Feb 29, 2024

@Deepali1211 there are no restrictions, the email address is extracted from your git commit.

@Deepali1211
Copy link
Contributor Author

Yes, understood the point. Thank you so much!

zhaohehuhu pushed a commit to zhaohehuhu/incubator-kyuubi that referenced this pull request Mar 21, 2024
@zhaohehuhu
[KYUUBI apache#6101] Bump Jackson from 2.15.0 to 2.15.4
75f974c 
# 🔍 Description
## Issue References 🔗

This pull request fixes apache#6101

## Describe Your Solution 🔧

This updates the jackson version from 2.15.0 to 2.15.4

## Types of changes 🔖

- [ ] Bugfix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)

## Test Plan 🧪

#### Behavior Without This Pull Request ⚰️

#### Behavior With This Pull Request 🎉

#### Related Unit Tests

---

# Checklist 📝

- [X] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)

**Be nice. Be informative.**

Closes apache#6110 from Deepali1211/upgrade-jackson.

Closes apache#6101

5da3a25 [Deepali Gupta] Upgrade jackson to 2.15.4
2666a1e [Deepali Gupta] Upgrade jackson to 2.15.4

Authored-by: Deepali Gupta <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pan3793 pan3793 pan3793 approved these changes

Assignees

@Deepali1211 Deepali1211

Labels
kind:build
Projects
None yet
Milestone
v1.9.0
Development

Successfully merging this pull request may close these issues.

Upgrade Jackson from 2.15.0 to 2.15.4
2 participants
@Deepali1211 @pan3793