xds: Remove verify TODO for onResult2 error status

ejona86 · ejona86 · commit 4995700069ad · 2025-09-19T08:23:51.000-07:00
This had been accidentally left in 0c179e3. Requesting a refresh is pretty close to RetryingNameResolver's behavior of exponential backoff. While not identical, it is the closest we can get easily.
diff --git a/xds/src/main/java/io/grpc/xds/XdsNameResolver.java b/xds/src/main/java/io/grpc/xds/XdsNameResolver.java
@@ -331,7 +331,6 @@ private void updateResolutionResult(XdsConfig xdsConfig) {
             .setServiceConfig(parsedServiceConfig)
             .build();
     if (!listener.onResult2(result).isOk()) {
-      // TODO: check if this is right
       resolveState.xdsDependencyManager.requestReresolution();
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -331,7 +331,6 @@ private void updateResolutionResult(XdsConfig xdsConfig) {`
`331`	`331`	`.setServiceConfig(parsedServiceConfig)`
`332`	`332`	`.build();`
`333`	`333`	`if (!listener.onResult2(result).isOk()) {`
`334`		`- // TODO: check if this is right`
`335`	`334`	`resolveState.xdsDependencyManager.requestReresolution();`
`336`	`335`	`}`
`337`	`336`	`}`