diff options
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; |