core: In PF, remove extraneous index.reset()

ejona86 · ejona86 · commit 9762945f813d · 2024-08-21T07:16:29.000-07:00
The index was just reset by updateGroups().
diff --git a/core/src/main/java/io/grpc/internal/PickFirstLeafLoadBalancer.java b/core/src/main/java/io/grpc/internal/PickFirstLeafLoadBalancer.java
@@ -130,9 +130,8 @@ public Status acceptResolvedAddresses(ResolvedAddresses resolvedAddresses) {
         SubchannelData subchannelData = subchannels.get(previousAddress);
         subchannelData.getSubchannel().updateAddresses(addressIndex.getCurrentEagAsList());
         return Status.OK;
-      } else {
-        addressIndex.reset(); // Previous ready subchannel not in the new list of addresses
       }
+      // Previous ready subchannel not in the new list of addresses
     } else {
       addressIndex.updateGroups(newImmutableAddressGroups);
     }

Original file line number	Diff line number	Diff line change
`@@ -130,9 +130,8 @@ public Status acceptResolvedAddresses(ResolvedAddresses resolvedAddresses) {`
`130`	`130`	`SubchannelData subchannelData = subchannels.get(previousAddress);`
`131`	`131`	`subchannelData.getSubchannel().updateAddresses(addressIndex.getCurrentEagAsList());`
`132`	`132`	`return Status.OK;`
`133`		`- } else {`
`134`		`- addressIndex.reset(); // Previous ready subchannel not in the new list of addresses`
`135`	`133`	`}`
	`134`	`+ // Previous ready subchannel not in the new list of addresses`
`136`	`135`	`} else {`
`137`	`136`	`addressIndex.updateGroups(newImmutableAddressGroups);`
`138`	`137`	`}`