resolve all the comments

CHANGELOG.md

@@ -15,7 +15,7 @@ and what APIs have changed, if applicable.
 ## [Unreleased]
 
 ## [29.58.7] - 2024-09-13
-Add WildcardResourceSubscriber for restli-resource-explorer subscribe to all resources
+- Add WildcardResourceSubscriber which could subscribe to all resources, like NODE and URIMap resources.
 
 ## [29.58.6] - 2024-09-08
 - Allow for null paging inside Collection response envelopes

d2/src/main/java/com/linkedin/d2/xds/XdsClientImpl.java

@@ -77,7 +77,7 @@ public class XdsClientImpl extends XdsClient
       Arrays.stream(ResourceType.values())
           .filter(e -> e.typeUrl() != null)
           .collect(Collectors.toMap(Function.identity(), e -> new HashMap<>())));
-  private final Map<ResourceType, WildcardResourceSubscriber> _wildcardSubscribers = new HashMap<>();
+  private final Map<ResourceType, WildcardResourceSubscriber> _wildcardSubscribers = Maps.newEnumMap(ResourceType.class);
   private final Node _node;
   private final ManagedChannel _managedChannel;
   private final ScheduledExecutorService _executorService;
@@ -861,6 +861,7 @@ public void run()
         {
           rewrittenType = type;
         }
+        // If there is a wildcard subscriber, we should always send a wildcard request to the server.
         if (getWildcardResourceSubscriber(type) != null)
         {
           resources.add("*");