This is an automated email from the ASF dual-hosted git repository. liubao pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-servicecomb-java-chassis.git
commit 54cd10152088d08b20ce295cc112dd9f511bb47a Author: heyile <[email protected]> AuthorDate: Sun Oct 14 12:06:36 2018 +0800 [SCB-837] make http2 production ready: set default value --- .../servicecomb/transport/rest/client/TransportClientConfig.java | 8 ++++++-- .../apache/servicecomb/transport/rest/vertx/TransportConfig.java | 3 ++- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/transports/transport-rest/transport-rest-client/src/main/java/org/apache/servicecomb/transport/rest/client/TransportClientConfig.java b/transports/transport-rest/transport-rest-client/src/main/java/org/apache/servicecomb/transport/rest/client/TransportClientConfig.java index a8ae6c6..1efd952 100644 --- a/transports/transport-rest/transport-rest-client/src/main/java/org/apache/servicecomb/transport/rest/client/TransportClientConfig.java +++ b/transports/transport-rest/transport-rest-client/src/main/java/org/apache/servicecomb/transport/rest/client/TransportClientConfig.java @@ -19,6 +19,8 @@ package org.apache.servicecomb.transport.rest.client; import com.netflix.config.DynamicPropertyFactory; +import io.vertx.core.http.HttpClientOptions; + public final class TransportClientConfig { private static Class<? extends RestTransportClient> restTransportClientCls = RestTransportClient.class; @@ -38,12 +40,14 @@ public final class TransportClientConfig { } public static int getHttp2ConnectionMaxPoolSize() { - return DynamicPropertyFactory.getInstance().getIntProperty("servicecomb.rest.client.http2.maxPoolSize", 1) + return DynamicPropertyFactory.getInstance().getIntProperty("servicecomb.rest.client.http2.maxPoolSize", + HttpClientOptions.DEFAULT_HTTP2_MAX_POOL_SIZE) .get(); } public static int getHttp2MultiplexingLimit() { - return DynamicPropertyFactory.getInstance().getIntProperty("servicecomb.rest.client.http2.multiplexingLimit", -1) + return DynamicPropertyFactory.getInstance().getIntProperty("servicecomb.rest.client.http2.multiplexingLimit", + HttpClientOptions.DEFAULT_HTTP2_MULTIPLEXING_LIMIT) .get(); } diff --git a/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/TransportConfig.java b/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/TransportConfig.java index c9cb96d..d3fd4f1 100644 --- a/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/TransportConfig.java +++ b/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/TransportConfig.java @@ -84,7 +84,8 @@ public final class TransportConfig { public static long getMaxConcurrentStreams() { return DynamicPropertyFactory.getInstance() - .getLongProperty("servicecomb.rest.server.http2.concurrentStreams", 100L) + .getLongProperty("servicecomb.rest.server.http2.concurrentStreams", + HttpServerOptions.DEFAULT_INITIAL_SETTINGS_MAX_CONCURRENT_STREAMS) .get(); }
