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

[ZOOKEEPER-4831] update slf4j from 1.x to 2.0.13, logback to 1.3.14 #2162

Merged
merged 2 commits into from May 11, 2024

Conversation

shoothzj
Copy link
Member

Update slf4j to 2.x, logback to 1.3.14. This is a backward-compatible change.

slf4j

SLF4J API version 2.0.0 requires Java 8 and introduces a backward-compatible fluent logging API. By backward-compatible, we mean that existing logging frameworks do not have to be changed in order for the user to benefit from the fluent logging API.

logback

Version 1.3.x supports Java EE, while version 1.4.x supports Jakarta EE. The two versions are feature identical.

Both 1.3.x and 1.4.x series require SLF4J 2.0.x or later.

The 1.3.x series requires Java 8 at runtime.

Signed-off-by: ZhangJian He <shoothzj@gmail.com>
Copy link
Member

@ctubbsii ctubbsii left a comment

Choose a reason for hiding this comment

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

👍 to updating the versions. I would suggest only doing this change in the next release, not in previous bugfixes for earlier branches.

I think some of the docs should be changed. Suggestions inline below.

Co-authored-by: Christopher Tubbs <ctubbsii@apache.org>
@shoothzj
Copy link
Member Author

shoothzj commented May 3, 2024

@ctubbsii Thanks for your review. Fixed.

to updating the versions. I would suggest only doing this change in the next release, not in previous bugfixes for earlier branches.

definitely.

@shoothzj
Copy link
Member Author

shoothzj commented May 9, 2024

@eolivelli @maoling @tisonkun PTAL, can we merge this into master branch

Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

LGTM

We need at least another binding +1

@tisonkun
Copy link
Member

tisonkun commented May 9, 2024

Thanks for your contribution!

@shoothzj
Copy link
Member Author

ping @eolivelli @maoling @tisonkun

@eolivelli eolivelli merged commit 66202cb into apache:master May 11, 2024
13 checks passed
@eolivelli
Copy link
Contributor

Merged thanks

@shoothzj shoothzj deleted the slf4j-update branch May 11, 2024 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants