[GitHub] [servicecomb-java-chassis] liubao68 commented on a change in pull request #1301: [SCB-1450] Microservice instance may get stuck when watch connection to sc is recovered and pulling instance a
liubao68 commented on a change in pull request #1301: [SCB-1450] Microservice instance may get stuck when watch connection to sc is recovered and pulling instance action is executed in eventloop thread URL: https://github.com/apache/servicecomb-java-chassis/pull/1301#discussion_r317398083 ## File path: service-registry/src/main/java/org/apache/servicecomb/serviceregistry/client/http/AbstractClientPool.java ## @@ -42,27 +45,41 @@ public static final String PROXY_KEY = "sc.consumer"; + private static final String EVENT_LOOP_POOL_SIZE = "servicecomb.service.registry.client.eventLoopPoolSize"; + Review comment: move all configurations to ServiceRegistryConfig is better 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [servicecomb-java-chassis] liubao68 commented on a change in pull request #1301: [SCB-1450] Microservice instance may get stuck when watch connection to sc is recovered and pulling instance a
liubao68 commented on a change in pull request #1301: [SCB-1450] Microservice instance may get stuck when watch connection to sc is recovered and pulling instance action is executed in eventloop thread URL: https://github.com/apache/servicecomb-java-chassis/pull/1301#discussion_r317397997 ## File path: service-registry/src/main/java/org/apache/servicecomb/serviceregistry/config/ServiceRegistryConfig.java ## @@ -109,22 +109,23 @@ public HttpVersion getHttpVersion() { return HttpVersion.valueOf(property.get()); } - public int getWorkerPoolSize() { -String workerPoolSizeKey = "servicecomb.service.registry.client.workerPoolSize"; + public int getInstances() { +String instances = "servicecomb.service.registry.client.instances "; Review comment: Define a constant is better 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services