fix p2c client bug

huanglongchao · huanglongchao · commit 98b90164ba74 · 2024-03-18T10:43:50.000+08:00
diff --git a/client/impl/src/main/java/com/alipay/sofa/registry/client/remoting/ClientConnection.java b/client/impl/src/main/java/com/alipay/sofa/registry/client/remoting/ClientConnection.java
@@ -130,7 +130,7 @@ private boolean connect() {
       try {
         // Power of Two Choices
         if (serverNodes.size() > 1) {
-          choosed = serverNodes.get(i).getWeight() > serverNodes.get(i + 1).getWeight() ? i + 1 : i;
+          choosed = serverNodes.get(i).getWeight() > serverNodes.get( (i + 1) % serverNodes.size() ).getWeight() ? (i + 1) % serverNodes.size() : i;
         }
         connection = connect(serverNodes.get(choosed));
         if (null != connection && connection.isFine()) {

Original file line number	Diff line number	Diff line change
`@@ -130,7 +130,7 @@ private boolean connect() {`
`130`	`130`	`try {`
`131`	`131`	`// Power of Two Choices`
`132`	`132`	`if (serverNodes.size() > 1) {`
`133`		`- choosed = serverNodes.get(i).getWeight() > serverNodes.get(i + 1).getWeight() ? i + 1 : i;`
	`133`	`+ choosed = serverNodes.get(i).getWeight() > serverNodes.get( (i + 1) % serverNodes.size() ).getWeight() ? (i + 1) % serverNodes.size() : i;`
`134`	`134`	`}`
`135`	`135`	`connection = connect(serverNodes.get(choosed));`
`136`	`136`	`if (null != connection && connection.isFine()) {`