aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorhcpp <hcpp@ydb.tech>2022-07-20 13:01:32 +0300
committerhcpp <hcpp@ydb.tech>2022-07-20 13:01:32 +0300
commitd4381e2b1f8dd1bd89561b49b36481a01843de6b (patch)
treef7e3e40cc6eee74ad6306c3c8c2094d58513b64a
parent4aab304cf9fba378a52a0bcfc2b5a9663f2300cf (diff)
downloadydb-d4381e2b1f8dd1bd89561b49b36481a01843de6b.tar.gz
fetcher owner id has been improved
-rw-r--r--ydb/core/yq/libs/actors/pending_fetcher.cpp20
1 files changed, 12 insertions, 8 deletions
diff --git a/ydb/core/yq/libs/actors/pending_fetcher.cpp b/ydb/core/yq/libs/actors/pending_fetcher.cpp
index ffbac05fccc..0162b2c8fdf 100644
--- a/ydb/core/yq/libs/actors/pending_fetcher.cpp
+++ b/ydb/core/yq/libs/actors/pending_fetcher.cpp
@@ -144,6 +144,7 @@ public:
, CredentialsFactory(credentialsFactory)
, S3Gateway(s3Gateway)
, PqCmConnections(std::move(pqCmConnections))
+ , FetcherGuid(CreateGuidAsString())
, ClientCounters(clientCounters)
, TenantName(tenantName)
, InternalServiceId(MakeInternalServiceActorId())
@@ -164,9 +165,7 @@ public:
DatabaseResolver = Register(CreateDatabaseResolver(MakeYqlAnalyticsHttpProxyId(), CredentialsFactory));
Send(SelfId(), new NActors::TEvents::TEvWakeup());
- TString guidActor = CreateGuidAsString();
- LOG_I("STARTED " + guidActor);
- LogScope.ConstructInPlace(NActors::TActivationContext::ActorSystem(), NKikimrServices::YQL_PROXY, guidActor);
+ LogScope.ConstructInPlace(NActors::TActivationContext::ActorSystem(), NKikimrServices::YQL_PROXY, FetcherGuid);
}
private:
@@ -240,10 +239,10 @@ private:
}
void GetPendingTask() {
- OwnerId = CreateGuidAsString();
- LOG_D("Request Private::GetTask" << ", Owner: " << OwnerId << ", Host: " << HostName() << ", Tenant: " << TenantName);
+ FetcherGeneration++;
+ LOG_D("Request Private::GetTask" << ", Owner: " << GetOwnerId() << ", Host: " << HostName() << ", Tenant: " << TenantName);
Yq::Private::GetTaskRequest request;
- request.set_owner_id(OwnerId);
+ request.set_owner_id(GetOwnerId());
request.set_host(HostName());
request.set_tenant(TenantName);
Send(InternalServiceId, new TEvInternalService::TEvGetTaskRequest(request));
@@ -256,6 +255,10 @@ private:
}
+ TString GetOwnerId() const {
+ return FetcherGuid + ToString(FetcherGeneration);
+ }
+
void RunTask(const Yq::Private::GetTaskResult::Task& task) {
LOG_D("NewTask:"
<< " Scope: " << task.scope()
@@ -317,7 +320,7 @@ private:
PrivateApiConfig, GatewaysConfig, PingerConfig,
task.text(), task.scope(), task.user_token(),
DatabaseResolver, queryId,
- task.user_id(), OwnerId, task.generation(),
+ task.user_id(), GetOwnerId(), task.generation(),
VectorFromProto(task.connection()),
VectorFromProto(task.binding()),
CredentialsFactory,
@@ -386,7 +389,8 @@ private:
const IHTTPGateway::TPtr S3Gateway;
const ::NPq::NConfigurationManager::IConnections::TPtr PqCmConnections;
- TString OwnerId;
+ const TString FetcherGuid;
+ uint64_t FetcherGeneration = 0;
const ::NMonitoring::TDynamicCounterPtr ClientCounters;
TMaybe<NYql::NLog::TScopedBackend<NYql::NDq::TYqlLogScope>> LogScope;