database/mariadb-columnstore-engine
1
0
Fork 0
mirror of https://github.com/mariadb-corporation/mariadb-columnstore-engine.git synced 2025-04-18 21:44:02 +03:00
Code

MCOL-3983: always set threadpool max threads in storagemanager (#2269)

Browse Source
This commit is contained in:
benthompson15 2022-02-23 12:21:43 -06:00 committed by GitHub
parent 0c978e252d
commit dc4f6865d1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
storage-manager/src/Downloader.cpp
View File

@ -214,7 +214,11 @@ void Downloader::configListener()
// Downloader threads
string stmp = Config::get()->getValue("ObjectStorage", "max_concurrent_downloads");
if (maxDownloads == 0)
{
maxDownloads = 20;
workers.setMaxThreads(maxDownloads);
logger->log(LOG_INFO, "max_concurrent_downloads = %u",maxDownloads);
}
if (stmp.empty())
{
logger->log(LOG_CRIT, "max_concurrent_downloads is not set. Using current value = %u", maxDownloads);

4
storage-manager/src/Synchronizer.cpp
View File

@ -884,7 +884,11 @@ void Synchronizer::configListener()
// Uploader threads
string stmp = Config::get()->getValue("ObjectStorage", "max_concurrent_uploads");
if (maxUploads == 0)
{
maxUploads = 20;
threadPool->setMaxThreads(maxUploads);
logger->log(LOG_INFO, "max_concurrent_uploads = %u",maxUploads);
}
if (stmp.empty())
{
logger->log(LOG_CRIT, "max_concurrent_uploads is not set. Using current value = %u", maxUploads);

6
storage-manager/storagemanager.cnf.in
View File

@ -55,9 +55,6 @@ journal_path = @ENGINE_DATADIR@/storagemanager/journal
# max_concurrent_downloads is what is sounds like, per node.
# This is not a global setting.
#
# The default was changed from 20 to 21 as a temporary workaround
# for a bug. It can be anything but 20.
max_concurrent_downloads = 21
# max_concurrent_uploads is what is sounds like, per node.
@ -66,9 +63,6 @@ max_concurrent_downloads = 21
# has low upstream bandwidth, consider lowering this value to the minimum
# necessary to saturate your network. This will reduce the latency of certain
# operations and improve your experience.
#
# The default was changed from 20 to 21 as a temporary workaround
# for a bug. It can be anything but 20.
max_concurrent_uploads = 21
# common_prefix_depth is the depth of the common prefix that all files