Make Ring Hash LB a petiole policy by larry-safran · Pull Request #10610 · grpc/grpc-java (original) (raw)

YifeiZhuang

YifeiZhuang

YifeiZhuang

YifeiZhuang

@larry-safran

@larry-safran

@larry-safran

@larry-safran

…ildren just to immediately delete them.

@larry-safran

@larry-safran

@larry-safran

@larry-safran

@larry-safran

@larry-safran

…irst 2 elements, but rather takes the first ring element not in TF and uses that.

Also, added some comments, fixed a couple of bugs in MultiChildLoadBalancer and replaced SubchannelView with direct usage of RingHashChildLbState.

@larry-safran

@larry-safran

@larry-safran

… created and respond to some code review comments.

@larry-safran

…es explicitly do the shutdowns after updating picker by default

@larry-safran

…shutdown in acceptResolvedAddresses.

@larry-safran

…state and removed child cleanup in acceptResolvedAddresses.

@larry-safran

…child cleanup when processing acceptResolvedAddresses.

@larry-safran

…y to start a connection in updateBalancingState when there aren't any active subchannels.

@larry-safran

@larry-safran

@larry-safran

…and there are 2+ TF then report TF. 1 TF + idle/connecting report connecting.

@larry-safran

@larry-safran

@ejona86 @larry-safran

The methods other than forTarget() were no longer used after 493af03. forTarget() was no longer used after cda0e9d.

@temawi @larry-safran

This class is of questionable utility and generally not used.

@larry-safran

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators

Feb 8, 2024

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})