diff options
| author | Kirill Pleshivtsev <[email protected]> | 2026-07-17 19:42:49 +0700 |
|---|---|---|
| committer | GitHub <[email protected]> | 2026-07-17 12:42:49 +0000 |
| commit | efbd3e7a1a2582153432fa87ba2f5a6ca5412bc3 (patch) | |
| tree | 90d92f56b9ac701ede42f9af313c2033f3796b94 | |
| parent | 2cda54717904055f1004962de8ff4d359740c50f (diff) | |
[NBS] Fix TPartitionActor immortality (#46912)
| -rw-r--r-- | ydb/core/nbs/cloud/blockstore/libs/storage/partition_direct/partition_direct_actor.cpp | 26 | ||||
| -rw-r--r-- | ydb/core/nbs/cloud/blockstore/libs/storage/partition_direct/partition_direct_actor.h | 1 |
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; |
