Skip to content

rls: Add route lookup reason to request#12442

Merged
kannanjgithub merged 10 commits intogrpc:masterfrom
kannanjgithub:rls-reason2
Nov 4, 2025
Merged

rls: Add route lookup reason to request#12442
kannanjgithub merged 10 commits intogrpc:masterfrom
kannanjgithub:rls-reason2

Conversation

@kannanjgithub
Copy link
Copy Markdown
Contributor

@kannanjgithub kannanjgithub commented Oct 24, 2025

b/348690462

@kannanjgithub kannanjgithub requested a review from ejona86 October 24, 2025 14:24
Comment thread rls/src/main/java/io/grpc/rls/CachingRlsLbClient.java Outdated
Comment thread rls/src/main/java/io/grpc/rls/RlsProtoData.java Outdated
Comment thread rls/src/main/java/io/grpc/rls/RlsRequestFactory.java Outdated
Comment thread rls/src/main/java/io/grpc/rls/CachingRlsLbClient.java Outdated
Comment thread rls/src/main/java/io/grpc/rls/CachingRlsLbClient.java Outdated
…fter a backed off entry is reprocessed, and the style related comments. Added assertions for lookup reason in unit tests.
…fter a backed off entry is reprocessed, and the style related comments. Added assertions for lookup reason in unit tests.
Comment thread rls/src/main/java/io/grpc/rls/CachingRlsLbClient.java Outdated
@ejona86 ejona86 added the TODO:backport PR needs to be backported. Removed after backport complete label Oct 31, 2025
@kannanjgithub kannanjgithub merged commit 76a1f2a into grpc:master Nov 4, 2025
14 of 17 checks passed
@kannanjgithub kannanjgithub deleted the rls-reason2 branch November 4, 2025 07:26
ejona86 pushed a commit to ejona86/grpc-java that referenced this pull request Nov 4, 2025
ejona86 pushed a commit that referenced this pull request Nov 5, 2025
@ejona86 ejona86 removed the TODO:backport PR needs to be backported. Removed after backport complete label Nov 5, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants