-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
xDS: Include orca named_metrics in LRS load reports #32690
Conversation
c3dbe42
to
005284c
Compare
src/core/ext/filters/client_channel/lb_policy/weighted_round_robin/weighted_round_robin.cc
Outdated
Show resolved
Hide resolved
src/core/ext/filters/client_channel/lb_policy/weighted_round_robin/weighted_round_robin.cc
Outdated
Show resolved
Hide resolved
12692ad
to
ba7e68a
Compare
Yousuk, I would ask again that you please not force-push to a branch once a review has started. This makes it very hard for the reviewer to see what has changed since the last review. |
src/core/ext/filters/client_channel/lb_policy/weighted_round_robin/weighted_round_robin.cc
Outdated
Show resolved
Hide resolved
The EPS PR #32657 has been rolled back, will hold until we roll it forward. |
@markdroth PR is now fully ready (pending tests). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
I removed arg name comments for the matcher instead of adding NOLINT()'s, it looks clang still cannot correctly handle MATCHER macros. |
<!-- If you know who should review your pull request, please assign it to that person, otherwise the pull request would get assigned randomly. If your pull request is for a specific language, please add the appropriate lang label. -->
<!-- If you know who should review your pull request, please assign it to that person, otherwise the pull request would get assigned randomly. If your pull request is for a specific language, please add the appropriate lang label. -->
No description provided.