Skip to content

Remove deprecation warnings. (#1613) #1615

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
Closed
wants to merge 1 commit into from
Closed

Remove deprecation warnings. (#1613) #1615

wants to merge 1 commit into from

Conversation

vbabanin
Copy link
Member

Backport of (#1613)

JAVA-5770

@vbabanin vbabanin requested a review from rozza January 28, 2025 06:31
@vbabanin vbabanin self-assigned this Jan 28, 2025
rozza
rozza approved these changes Jan 28, 2025
View reviewed changes
Copy link
Member

@rozza rozza left a comment

Choose a reason for hiding this comment

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

LGTM

@jyemin
Copy link
Collaborator

jyemin commented Jan 30, 2025

@vbabanin @rozza except for critical security issues we generally don't backport fixes past the last minor release, so not sure the intention of merging this into the 5.2 branch. Is it already in the 5.3 branch?

@rozza
Copy link
Member

rozza commented Jan 30, 2025

@jyemin I think the log warnings were erroneously added to 5.2.x

@jyemin
Copy link
Collaborator

jyemin commented Jan 30, 2025

Yes, I think so too, but still, we generally only patch the latest minor, which is now 5.3.

@vbabanin vbabanin closed this Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rozza rozza rozza approved these changes

Assignees

@vbabanin vbabanin

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vbabanin @jyemin @rozza