[GitHub] [incubator-doris] chaoyli commented on a change in pull request #4808: [optimize] optimize default value for thriftserver's config key "thrift_client_timeout_ms"

[GitBox <gi...@apache.org>]

chaoyli commented on a change in pull request #4808:
URL: https://github.com/apache/incubator-doris/pull/4808#discussion_r513143370



##########
File path: fe/fe-core/src/main/java/org/apache/doris/common/Config.java
##########
@@ -318,8 +318,10 @@
      * The connection timeout and socket timeout config for thrift server
      * The value for thrift_client_timeout_ms is set to be larger than zero to prevent
      * some hang up problems in java.net.SocketInputStream.socketRead0
+     * 
+     * make it one hour to avoid fe close the socket so frequently, that be will retry to cost more time
      */
-    @ConfField public static int thrift_client_timeout_ms = 30000;

Review comment:
       when will fe close the socket frequently?




----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org