Sort added router list before set the 'routers' field of the RouterChain (#3969)

tswstarplanet · kun-song · commit 392fcbfd1a51 · 2019-05-07T08:05:09.000+08:00
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
@@ -79,7 +79,7 @@ public void addRouters(List<Router> routers) {
         List<Router> newRouters = new ArrayList<>();
         newRouters.addAll(builtinRouters);
         newRouters.addAll(routers);
-        CollectionUtils.sort(routers);
+        CollectionUtils.sort(newRouters);
         this.routers = newRouters;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ public void addRouters(List<Router> routers) {`
`79`	`79`	`List<Router> newRouters = new ArrayList<>();`
`80`	`80`	`newRouters.addAll(builtinRouters);`
`81`	`81`	`newRouters.addAll(routers);`
`82`		`- CollectionUtils.sort(routers);`
	`82`	`+ CollectionUtils.sort(newRouters);`
`83`	`83`	`this.routers = newRouters;`
`84`	`84`	`}`
`85`	`85`