Skip to content

Upgrade ZooKeeper from 3.7.1 to 3.8.0#2920

Merged
ctubbsii merged 1 commit intoapache:mainfrom
dlmarion:upgrade-zk-380
Sep 16, 2022
Merged

Upgrade ZooKeeper from 3.7.1 to 3.8.0#2920
ctubbsii merged 1 commit intoapache:mainfrom
dlmarion:upgrade-zk-380

Conversation

@dlmarion
Copy link
Copy Markdown
Contributor

@dlmarion dlmarion commented Sep 8, 2022

No description provided.

@dlmarion dlmarion self-assigned this Sep 8, 2022
@dlmarion
Copy link
Copy Markdown
Contributor Author

dlmarion commented Sep 8, 2022

Kicked off full IT build

@dlmarion dlmarion requested a review from ctubbsii September 8, 2022 19:24
@dlmarion
Copy link
Copy Markdown
Contributor Author

dlmarion commented Sep 9, 2022

Full IT build passed

Copy link
Copy Markdown
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.

This is fine. Related change: we should consider using SLF4J 2, which uses the Java service loader to load the transitive logging dependencies at runtime, rather than the StaticLoggerBinder that SLF4J 1.7 used. It might save some trouble with transitive deps.

@ctubbsii ctubbsii merged commit 82a3071 into apache:main Sep 16, 2022
@dlmarion dlmarion deleted the upgrade-zk-380 branch November 1, 2022 15:38
@ctubbsii ctubbsii added this to the 2.1.0 milestone Jul 12, 2024
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