Merge pull request #2623 from murgatroid99/grpc-js_pick_first_reset_listener_fix

grpc-js: pick_first: fix currentPick comparison in resetSubchannelList

Author: murgatroid99

packages/grpc-js/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@grpc/grpc-js",
-  "version": "1.9.11",
+  "version": "1.9.12",
   "description": "gRPC Library for Node - pure JS implementation",
   "homepage": "https://grpc.io/",
   "repository": "https://github.com/grpc/grpc-node/tree/master/packages/grpc-js",

packages/grpc-js/src/load-balancer-pick-first.ts

@@ -382,7 +382,7 @@ export class PickFirstLoadBalancer implements LoadBalancer {
 
   private resetSubchannelList() {
     for (const child of this.children) {
-      if (child.subchannel !== this.currentPick) {
+      if (!(this.currentPick && child.subchannel.realSubchannelEquals(this.currentPick))) {
         /* The connectivity state listener is the same whether the subchannel
          * is in the list of children or it is the currentPick, so if it is in
          * both, removing it here would cause problems. In particular, that