[incubator-dubbo] branch master updated: Sort added router list before set the 'routers' field of the RouterChain (#3969)

[songkun]

This is an automated email from the ASF dual-hosted git repository.

songkun pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-dubbo.git


The following commit(s) were added to refs/heads/master by this push:
     new 392fcbf  Sort added router list before set the 'routers' field of the 
RouterChain (#3969)
392fcbf is described below

commit 392fcbfd1a51418c92dce096b2d466c5890b9a17
Author: Taosheng Wei <tswstarpla...@apache.org>
AuthorDate: Tue May 7 08:05:09 2019 +0800

    Sort added router list before set the 'routers' field of the RouterChain 
(#3969)
---
 .../src/main/java/org/apache/dubbo/rpc/cluster/RouterChain.java         | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/RouterChain.java 
b/dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/RouterChain.java
index 7586afe..8ba22b1 100644
--- a/dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/RouterChain.java
+++ b/dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/RouterChain.java
@@ -79,7 +79,7 @@ public class RouterChain<T> {
         List<Router> newRouters = new ArrayList<>();
         newRouters.addAll(builtinRouters);
         newRouters.addAll(routers);
-        CollectionUtils.sort(routers);
+        CollectionUtils.sort(newRouters);
         this.routers = newRouters;
     }