summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorkomels <[email protected]>2023-08-17 14:03:48 +0300
committerkomels <[email protected]>2023-08-17 17:02:33 +0300
commit94b610e28ef001c9db9530909402f40fff14b356 (patch)
treeb268383c75b7cb5eef063a7e5eda87cfc43f1aaf
parent8ccea506f6a55e30737068f23d49341bb25c233d (diff)
Bugfix
-rw-r--r--ydb/services/persqueue_v1/actors/schema_actors.cpp37
1 files changed, 12 insertions, 25 deletions
diff --git a/ydb/services/persqueue_v1/actors/schema_actors.cpp b/ydb/services/persqueue_v1/actors/schema_actors.cpp
index c3ef4e28cb1..99a2e3b0ee6 100644
--- a/ydb/services/persqueue_v1/actors/schema_actors.cpp
+++ b/ydb/services/persqueue_v1/actors/schema_actors.cpp
@@ -614,21 +614,13 @@ void TDescribeTopicActorImpl::RequestBalancer(const TActorContext& ctx) {
if (Settings.RequireLocation && !PendingLocation && !GotLocation) {
return RequestPartitionsLocationIfRequired(ctx);
}
- switch (Settings.Mode) {
- case TDescribeTopicActorSettings::EMode::DescribeConsumer:
- case TDescribeTopicActorSettings::EMode::DescribeTopic:
- if (Settings.RequireStats) {
- NTabletPipe::SendData(
- ctx, *BalancerPipe,
- new TEvPersQueue::TEvGetReadSessionsInfo(NPersQueue::ConvertNewConsumerName(Settings.Consumer, ctx))
- );
- LOG_DEBUG_S(ctx, NKikimrServices::PQ_READ_PROXY, "DescribeTopicImpl " << ctx.SelfID.ToString() << ": Request sessions");
- ++RequestsInfly;
- }
- break;
- case TDescribeTopicActorSettings::EMode::DescribePartitions: {
- break;
- }
+ if (Settings.Mode == TDescribeTopicActorSettings::EMode::DescribeConsumer && Settings.RequireStats) {
+ NTabletPipe::SendData(
+ ctx, *BalancerPipe,
+ new TEvPersQueue::TEvGetReadSessionsInfo(NPersQueue::ConvertNewConsumerName(Settings.Consumer, ctx))
+ );
+ LOG_DEBUG_S(ctx, NKikimrServices::PQ_READ_PROXY, "DescribeTopicImpl " << ctx.SelfID.ToString() << ": Request sessions");
+ ++RequestsInfly;
}
}
@@ -746,16 +738,11 @@ void TDescribeTopicActorImpl::RequestAdditionalInfo(const TActorContext& ctx) {
}
void TDescribeTopicActorImpl::CheckCloseBalancerPipe(const TActorContext& ctx) {
- switch (Settings.Mode) {
- case TDescribeTopicActorSettings::EMode::DescribePartitions:
- if (RequestsInfly || PendingLocation)
- return;
- // no break;
- default:
- NTabletPipe::CloseClient(ctx, *BalancerPipe);
- *BalancerPipe = TActorId{};
- BalancerTabletId = 0;
- }
+ if (RequestsInfly || PendingLocation)
+ return;
+ NTabletPipe::CloseClient(ctx, *BalancerPipe);
+ *BalancerPipe = TActorId{};
+ BalancerTabletId = 0;
}