diff options
author | Juanli Shen <juanlishen@google.com> | 2018-07-13 19:52:59 -0700 |
---|---|---|
committer | Juanli Shen <juanlishen@google.com> | 2018-07-13 19:52:59 -0700 |
commit | be40b0d3a8cf2e37c80b2c248111051fa8bdf7bc (patch) | |
tree | 79384a0045d5bbc550666f047d088f9582755086 /src/proto | |
parent | f9f5c67aff91e4ad26371b0a2482a5011ab45226 (diff) |
Add server load reporting service
Diffstat (limited to 'src/proto')
-rw-r--r-- | src/proto/grpc/lb/v1/load_reporter.proto | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/proto/grpc/lb/v1/load_reporter.proto b/src/proto/grpc/lb/v1/load_reporter.proto index c2e4f23f6e..d57a37fed7 100644 --- a/src/proto/grpc/lb/v1/load_reporter.proto +++ b/src/proto/grpc/lb/v1/load_reporter.proto @@ -114,6 +114,10 @@ message Load { // num_calls_in_progress is the only valid entry. If in_progress_report is not // set, num_calls_in_progress will be ignored. If in_progress_report is set, // fields other than num_calls_in_progress and orphaned_load will be ignored. + // TODO(juanlishen): A Load is either an in_progress_report or not. We should + // make this explicit in hierarchy. From the log, I see in_progress_report_ + // has a random num_calls_in_progress_ when not set, which might lead to bug + // when the balancer process the load report. oneof in_progress_report { // The number of calls in progress (instantaneously) per load balancer id. int64 num_calls_in_progress = 5; |