imay commented on a change in pull request #3703:
URL: https://github.com/apache/incubator-doris/pull/3703#discussion_r431532217



##########
File path: be/src/common/config.h
##########
@@ -137,7 +140,7 @@ namespace config {
     // the maximum number of bytes to display on the debug webserver's log page
     CONF_Int64(web_log_bytes, "1048576");
     // number of threads available to serve backend execution requests
-    CONF_Int32(be_service_threads, "64");
+    CONF_Int32(be_service_threads, "512");

Review comment:
       I think 512 is some big, why not keep it 64?
   For most scenario, 64 is enough.

##########
File path: be/src/common/config.h
##########
@@ -271,7 +274,7 @@ namespace config {
     // Port to start debug webserver on
     CONF_Int32(webserver_port, "8040");
     // Number of webserver workers
-    CONF_Int32(webserver_num_workers, "5");
+    CONF_Int32(webserver_num_workers, "48");

Review comment:
       48 is some big, how about 12?




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



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

Reply via email to