diff options
author | hcpp <hcpp@ydb.tech> | 2022-07-20 13:01:32 +0300 |
---|---|---|
committer | hcpp <hcpp@ydb.tech> | 2022-07-20 13:01:32 +0300 |
commit | d4381e2b1f8dd1bd89561b49b36481a01843de6b (patch) | |
tree | f7e3e40cc6eee74ad6306c3c8c2094d58513b64a | |
parent | 4aab304cf9fba378a52a0bcfc2b5a9663f2300cf (diff) | |
download | ydb-d4381e2b1f8dd1bd89561b49b36481a01843de6b.tar.gz |
fetcher owner id has been improved
-rw-r--r-- | ydb/core/yq/libs/actors/pending_fetcher.cpp | 20 |
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; |