File tree Expand file tree Collapse file tree 1 file changed +0
-11
lines changed
xds/internal/balancer/ringhash Expand file tree Collapse file tree 1 file changed +0
-11
lines changed Original file line number Diff line number Diff line change @@ -231,17 +231,6 @@ func (b *ringhashBalancer) updatePickerLocked() {
231231 // TF. Since there must be at least one endpoint attempting to connect,
232232 // we need to trigger one.
233233 //
234- // TODO: https://github.com/grpc/grpc-go/issues/8085 - Restrict the
235- // condition under which an endpoint is connected. The pseudocode
236- // mentioned in A61 doesn't handle the following edge cases where the
237- // aggregated state is TF, but no endpoint actually enters TF:
238- // 1. There are four endpoints in the following states: TF, TF, READY,
239- // and IDLE. If the READY endpoint fails, it transitions to IDLE,
240- // resulting in the new states: TF, TF, IDLE, IDLE.
241- // 2. There are four endpoints in the following states: TF, TF,
242- // CONNECTING, and IDLE. If the CONNECTING endpoint is removed, the
243- // new states become: TF, TF, IDLE.
244-
245234 // After calling `ExitIdle` on a child balancer, the child will send a
246235 // picker update asynchronously. A race condition may occur if another
247236 // picker update from endpointsharding arrives before the child's
You can’t perform that action at this time.
0 commit comments