Skip to content

Commit 78b6755

Browse files
committed
Added IsWorkloadServiceRequired function
1 parent b501340 commit 78b6755

File tree

3 files changed

+13
-1
lines changed

3 files changed

+13
-1
lines changed

ydb/core/kqp/proxy_service/kqp_proxy_service.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1590,7 +1590,7 @@ class TKqpProxyService : public TActorBootstrapped<TKqpProxyService> {
15901590
}
15911591

15921592
const auto& poolConfig = poolInfo->Config;
1593-
if (poolConfig.ConcurrentQueryLimit == -1 && poolConfig.DatabaseLoadCpuThreshold < 0.0 && !poolConfig.QueryCancelAfter) {
1593+
if (!NWorkload::IsWorkloadServiceRequired(poolConfig)) {
15941594
ev->Get()->SetPoolConfig(poolConfig);
15951595
}
15961596

ydb/core/kqp/workload_service/kqp_workload_service.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -538,6 +538,10 @@ class TKqpWorkloadService : public TActorBootstrapped<TKqpWorkloadService> {
538538

539539
} // anonymous namespace
540540

541+
bool IsWorkloadServiceRequired(const NResourcePool::TPoolSettings& config) {
542+
return config.ConcurrentQueryLimit != -1 || config.DatabaseLoadCpuThreshold >= 0.0 || config.QueryCancelAfter;
543+
}
544+
541545
} // namespace NWorkload
542546

543547
IActor* CreateKqpWorkloadService(NMonitoring::TDynamicCounterPtr counters) {
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,18 @@
11
#pragma once
22

3+
#include <ydb/core/resource_pools/resource_pool_settings.h>
4+
35
#include <ydb/library/actors/core/actor.h>
46

57

68
namespace NKikimr::NKqp {
79

10+
namespace NWorkload {
11+
12+
bool IsWorkloadServiceRequired(const NResourcePool::TPoolSettings& config);
13+
14+
} // namespace NWorkload
15+
816
NActors::IActor* CreateKqpWorkloadService(NMonitoring::TDynamicCounterPtr counters);
917

1018
} // namespace NKikimr::NKqp

0 commit comments

Comments
 (0)