grpc-js: pick-first: Fix short circuit READY subchannel handling

This commit is contained in:
Michael Lumish 2024-07-08 15:08:30 -07:00
parent e804ad65b6
commit a5fac6f056
1 changed files with 2 additions and 4 deletions

View File

@ -485,6 +485,8 @@ export class PickFirstLoadBalancer implements LoadBalancer {
trace('connectToAddressList([' + addressList.map(address => subchannelAddressToString(address)) + '])');
for (const { subchannel } of newChildrenList) {
if (subchannel.getConnectivityState() === ConnectivityState.READY) {
this.channelControlHelper.addChannelzChild(subchannel.getChannelzRef());
subchannel.addConnectivityStateListener(this.subchannelStateListener);
this.pickSubchannel(subchannel);
return;
}
@ -500,10 +502,6 @@ export class PickFirstLoadBalancer implements LoadBalancer {
this.children = newChildrenList;
for (const { subchannel } of this.children) {
subchannel.addConnectivityStateListener(this.subchannelStateListener);
if (subchannel.getConnectivityState() === ConnectivityState.READY) {
this.pickSubchannel(subchannel);
return;
}
}
for (const child of this.children) {
if (