[GitHub] [incubator-doris] francisoliverlee 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>]

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



##########
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:
       in our case, query information_schema.tables.  i find fe would send a TCP RST command(try tcpdump 9020 in fe) to be to close the socket becasuse of thrift_client_timeout_ms is meet, the be log comes "no more data to read", the be would retry in 1s(default).




----------------------------------------------------------------
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