acelyc111 commented on a change in pull request #5158: URL: https://github.com/apache/incubator-doris/pull/5158#discussion_r549600492
########## File path: be/src/common/config.h ########## @@ -482,8 +422,8 @@ CONF_mInt64(write_buffer_size, "104857600"); // NOTICE(cmy): set these default values very large because we don't want to // impact the load performance when user upgrading Doris. // user should set these configs properly if necessary. -CONF_Int64(load_process_max_memory_limit_bytes, "107374182400"); // 100GB -CONF_Int32(load_process_max_memory_limit_percent, "80"); // 80% +CONF_mInt64(load_process_max_memory_limit_bytes, "107374182400"); // 100GB +CONF_mInt32(load_process_max_memory_limit_percent, "80"); // 80% Review comment: These 2 configs are only changed in LoadChannelMgr's init function, which will be called only once when server start, they are immutable. ---------------------------------------------------------------- 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