get service instances by Flux.blockLast() to resolve concurrent problem (#764)

CHANGELOG.md

@@ -18,3 +18,4 @@
 - [fix:fix discovery junit.](https://github.com/Tencent/spring-cloud-tencent/pull/730)
 - [adapt polaris-java 1.10.1 version](https://github.com/Tencent/spring-cloud-tencent/pull/746)
 - [Optimize: change RouteArgument.buildCustom to RouteArgument.fromLabel](https://github.com/Tencent/spring-cloud-tencent/pull/749)
+- [Optimize: get service instances by Flux.blockLast() to resolve concurrent problem](https://github.com/Tencent/spring-cloud-tencent/pull/764)

spring-cloud-tencent-polaris-loadbalancer/src/main/java/com/tencent/cloud/polaris/loadbalancer/LoadBalancerUtils.java

@@ -18,10 +18,8 @@
 
 package com.tencent.cloud.polaris.loadbalancer;
 
-import java.util.Collections;
+import java.util.ArrayList;
 import java.util.List;
-import java.util.concurrent.atomic.AtomicReference;
-import java.util.stream.Collectors;
 
 import com.tencent.cloud.common.metadata.MetadataContext;
 import com.tencent.cloud.common.pojo.PolarisServiceInstance;
@@ -42,13 +40,43 @@ import org.springframework.util.CollectionUtils;
  */
 public final class LoadBalancerUtils {
 
+	private static final int DEFAULT_WEIGHT = 100;
+
 	private LoadBalancerUtils() {
 	}
 
+	/**
+	 * transfer servers to ServiceInstances.
+	 *
+	 * @param servers servers
+	 * @return ServiceInstances
+	 */
 	public static ServiceInstances transferServersToServiceInstances(Flux<List<ServiceInstance>> servers) {
-		AtomicReference<List<Instance>> instances = new AtomicReference<>();
-		servers.subscribe(serviceInstances -> {
-			instances.set(serviceInstances.stream().map(serviceInstance -> {
+		List<ServiceInstance> serviceInstances = servers.blockLast();
+
+		List<Instance> instances = new ArrayList<>();
+		if (!CollectionUtils.isEmpty(serviceInstances)) {
+			for (ServiceInstance serviceInstance : serviceInstances) {
+				instances.add(transferServerToServiceInstance(serviceInstance));
+			}
+		}
+
+		String serviceName = null;
+		if (!CollectionUtils.isEmpty(instances)) {
+			serviceName = instances.get(0).getService();
+		}
+
+		ServiceKey serviceKey = new ServiceKey(MetadataContext.LOCAL_NAMESPACE, serviceName);
+		return new DefaultServiceInstances(serviceKey, instances);
+	}
+
+	/**
+	 * transfer ServiceInstance to DefaultInstance.
+	 *
+	 * @param serviceInstance serviceInstance
+	 * @return defaultInstance
+	 */
+	public static DefaultInstance transferServerToServiceInstance(ServiceInstance serviceInstance) {
 		DefaultInstance instance = new DefaultInstance();
 		instance.setNamespace(MetadataContext.LOCAL_NAMESPACE);
 		instance.setService(serviceInstance.getServiceId());
@@ -56,7 +84,7 @@ public final class LoadBalancerUtils {
 		instance.setId(serviceInstance.getInstanceId());
 		instance.setHost(serviceInstance.getHost());
 		instance.setPort(serviceInstance.getPort());
-				instance.setWeight(100);
+		instance.setWeight(DEFAULT_WEIGHT);
 		instance.setMetadata(serviceInstance.getMetadata());
 
 		if (serviceInstance instanceof PolarisServiceInstance) {
@@ -67,19 +95,5 @@ public final class LoadBalancerUtils {
 		}
 
 		return instance;
-			}).collect(Collectors.toList()));
-		});
-
-		String serviceName = null;
-		if (CollectionUtils.isEmpty(instances.get())) {
-			instances.set(Collections.emptyList());
-		}
-		else {
-			serviceName = instances.get().get(0).getService();
-		}
-
-		ServiceKey serviceKey = new ServiceKey(MetadataContext.LOCAL_NAMESPACE, serviceName);
-
-		return new DefaultServiceInstances(serviceKey, instances.get());
 	}
 }