[GitHub] [dubbo] AlbumenJ commented on a change in pull request #9424: [3.0] Refactor state router iterator to router chain

[GitBox <gi...@apache.org>]

AlbumenJ commented on a change in pull request #9424:
URL: https://github.com/apache/dubbo/pull/9424#discussion_r774914187



##########
File path: dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/RouterChain.java
##########
@@ -63,19 +64,16 @@
      */
     private volatile List<Router> builtinRouters = Collections.emptyList();
 
-    private volatile List<StateRouter<T>> builtinStateRouters = Collections.emptyList();
-    private volatile List<StateRouter<T>> stateRouters = Collections.emptyList();
+    private volatile StateRouter<T> headStateRouter;
+
+    private volatile List<StateRouter<T>> stateRouters;

Review comment:
       `stateRouters` is for unit test and qos management perpose.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org