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

[pick_first] implement address interleaving for Happy Eyeballs #34615

Merged

Conversation

markdroth
Copy link
Member

@markdroth markdroth commented Oct 6, 2023

No description provided.

@grpc-checks grpc-checks bot added bloat/low and removed bloat/none labels Oct 9, 2023
@markdroth markdroth changed the title [pick_first] implement address interleaving for Happy Eyeballs [pick_first] implement address interleaving for Happy Eyeballs Oct 11, 2023
@markdroth markdroth changed the title [pick_first] implement address interleaving for Happy Eyeballs [pick_first] implement address interleaving for Happy Eyeballs Oct 11, 2023
@markdroth markdroth marked this pull request as ready for review October 11, 2023 15:29
@markdroth markdroth merged commit 067fc48 into grpc:master Oct 12, 2023
70 of 71 checks passed
@markdroth markdroth deleted the dualstack_happy_eyeballs_interleaving branch October 12, 2023 17:21
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Oct 12, 2023
markdroth added a commit that referenced this pull request Oct 26, 2023
The original logic from #34615 was incorrect in cases where one address
family has a different number of addresses than the other(s).

Fixes b/307937051.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bloat/low imported Specifies if the PR has been imported to the internal repository lang/core per-call-memory/neutral per-channel-memory/neutral release notes: no Indicates if PR should not be in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants