Skip to content

Conversation

@Tim-Brooks
Copy link
Contributor

This reverts commit 43e3e24.

Copy link
Contributor

@ldematte ldematte left a comment

Choose a reason for hiding this comment

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

LGTM (clean backport of #120934)

@mhl-b
Copy link
Contributor

mhl-b commented Jan 28, 2025

There is a proper fix #121095. Maybe this backport is not needed.

@ldematte
Copy link
Contributor

ldematte commented Jan 29, 2025

There is a proper fix #121095. Maybe this backport is not needed.

👍 happy to discard this!
@mhl-b I think you can close this PR once #121095 is backported. Wdyt?

@mhl-b
Copy link
Contributor

mhl-b commented Jan 29, 2025

I can close this one, no problem.

@mhl-b
Copy link
Contributor

mhl-b commented Feb 1, 2025

Fixed #121095 and backported #121255

@mhl-b mhl-b closed this Feb 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport :Distributed Coordination/Network Http and internode communication implementations >non-issue v8.19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants