summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGeir Storli <geirst@verizonmedia.com>2021-09-08 15:28:02 +0000
committerGeir Storli <geirst@verizonmedia.com>2021-09-08 15:28:02 +0000
commit7739817edb2efdd9ac07203100038a454c6b6343 (patch)
treef45e2f5915209a35ddf297c84a51eeb31b3c7660
parent03a4dee06316630e954843f85d4f4958e7ac4210 (diff)
Increase tick wait duration for main distributor thread when running with multiple stripes.
This because it will no longer be running background maintenance jobs (non-event tick will instead be used primarily for resending full bucket fetches etc).
-rw-r--r--storage/src/vespa/storage/storageserver/distributornode.cpp4
-rw-r--r--storageframework/src/vespa/storageframework/generic/thread/tickingthread.h1
2 files changed, 4 insertions, 1 deletions
diff --git a/storage/src/vespa/storage/storageserver/distributornode.cpp b/storage/src/vespa/storage/storageserver/distributornode.cpp
index 0d129ffdb28..620bd3571ce 100644
--- a/storage/src/vespa/storage/storageserver/distributornode.cpp
+++ b/storage/src/vespa/storage/storageserver/distributornode.cpp
@@ -26,7 +26,9 @@ DistributorNode::DistributorNode(
: StorageNode(configUri, context, generationFetcher,
std::make_unique<HostInfo>(),
!communicationManager ? NORMAL : SINGLE_THREADED_TEST_MODE),
- _threadPool(framework::TickingThreadPool::createDefault("distributor")),
+ // TODO STRIPE: Change waitTime default to 100ms when legacy mode is removed.
+ _threadPool(framework::TickingThreadPool::createDefault("distributor",
+ (num_distributor_stripes > 0) ? 100ms : 5ms)),
_stripe_pool(std::make_unique<distributor::DistributorStripePool>()),
_context(context),
_timestamp_mutex(),
diff --git a/storageframework/src/vespa/storageframework/generic/thread/tickingthread.h b/storageframework/src/vespa/storageframework/generic/thread/tickingthread.h
index 0649d914c75..c80ba0e8945 100644
--- a/storageframework/src/vespa/storageframework/generic/thread/tickingthread.h
+++ b/storageframework/src/vespa/storageframework/generic/thread/tickingthread.h
@@ -78,6 +78,7 @@ struct ThreadLock {
struct TickingThreadPool : public ThreadLock {
using UP = std::unique_ptr<TickingThreadPool>;
+ // TODO STRIPE: Change waitTime default to 100ms when legacy mode is removed.
static TickingThreadPool::UP createDefault(
vespalib::stringref name,
vespalib::duration waitTime = 5ms,