summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKirill Pleshivtsev <[email protected]>2026-07-17 19:42:49 +0700
committerGitHub <[email protected]>2026-07-17 12:42:49 +0000
commitefbd3e7a1a2582153432fa87ba2f5a6ca5412bc3 (patch)
tree90d92f56b9ac701ede42f9af313c2033f3796b94
parent2cda54717904055f1004962de8ff4d359740c50f (diff)
[NBS] Fix TPartitionActor immortality (#46912)
-rw-r--r--ydb/core/nbs/cloud/blockstore/libs/storage/partition_direct/partition_direct_actor.cpp26
-rw-r--r--ydb/core/nbs/cloud/blockstore/libs/storage/partition_direct/partition_direct_actor.h1
2 files changed, 16 insertions, 11 deletions
diff --git a/ydb/core/nbs/cloud/blockstore/libs/storage/partition_direct/partition_direct_actor.cpp b/ydb/core/nbs/cloud/blockstore/libs/storage/partition_direct/partition_direct_actor.cpp
index 2bba24e8b5a..07d185ad5bf 100644
--- a/ydb/core/nbs/cloud/blockstore/libs/storage/partition_direct/partition_direct_actor.cpp
+++ b/ydb/core/nbs/cloud/blockstore/libs/storage/partition_direct/partition_direct_actor.cpp
@@ -46,22 +46,19 @@ TPartitionActor::TPartitionActor(
LOG_INFO(
NActors::TActivationContext::AsActorContext(),
NKikimrServices::NBS_PARTITION,
- "%s TPartitionActor: initialization started",
+ "%s initialization started",
LogTitle.GetWithTime().c_str());
}
TPartitionActor::~TPartitionActor() = default;
-void TPartitionActor::PassAway()
+void TPartitionActor::OnDetach(const TActorContext& ctx)
{
LOG_INFO(
NActors::TActivationContext::AsActorContext(),
NKikimrServices::NBS_PARTITION,
- "TPartitionActor: before detach");
-}
-
-void TPartitionActor::OnDetach(const TActorContext& ctx)
-{
+ "%s OnDetach",
+ LogTitle.GetWithTime().c_str());
Die(ctx);
}
@@ -70,6 +67,13 @@ void TPartitionActor::OnTabletDead(
const TActorContext& ctx)
{
Y_UNUSED(ev);
+
+ LOG_INFO(
+ NActors::TActivationContext::AsActorContext(),
+ NKikimrServices::NBS_PARTITION,
+ "%s OnTabletDead",
+ LogTitle.GetWithTime().c_str());
+
Die(ctx);
}
@@ -640,11 +644,13 @@ STFUNC(TPartitionActor::StateWork)
default:
if (!HandleDefaultEvents(ev, SelfId())) {
- LOG_DEBUG_S(
+ LOG_ERROR(
TActivationContext::AsActorContext(),
NKikimrServices::NBS_PARTITION,
- "Unhandled event type: " << ev->GetTypeRewrite()
- << " event: " << ev->ToString());
+ "%s Unhandled event type: %u event %s ",
+ LogTitle.GetWithTime().c_str(),
+ ev->GetTypeRewrite(),
+ ev->ToString().c_str());
}
break;
}
diff --git a/ydb/core/nbs/cloud/blockstore/libs/storage/partition_direct/partition_direct_actor.h b/ydb/core/nbs/cloud/blockstore/libs/storage/partition_direct/partition_direct_actor.h
index 12c3fbc44f8..da496076412 100644
--- a/ydb/core/nbs/cloud/blockstore/libs/storage/partition_direct/partition_direct_actor.h
+++ b/ydb/core/nbs/cloud/blockstore/libs/storage/partition_direct/partition_direct_actor.h
@@ -74,7 +74,6 @@ public:
NKikimr::TTabletStorageInfo* info);
~TPartitionActor() override;
- void PassAway() override;
static constexpr ui32 LogComponent = NKikimrServices::NBS_PARTITION;
using TCounters = TPartitionCounters;