Skip to content

Commit

Permalink
fix splits signal (ydb-platform#7273)
Browse files Browse the repository at this point in the history
  • Loading branch information
ssmike authored Jul 31, 2024
1 parent 3c1690d commit bd81a67
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion ydb/core/kqp/runtime/kqp_read_actor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,7 @@ class TKqpReadActor : public TActorBootstrapped<TKqpReadActor>, public NYql::NDq

TMaybe<ui32> NodeId = {};
bool IsFirst = false;
bool IsFake = false;


TShardState(ui64 tabletId)
Expand Down Expand Up @@ -439,6 +440,7 @@ class TKqpReadActor : public TActorBootstrapped<TKqpReadActor>, public NYql::NDq
CA_LOG_D("Shards State: " << state.ToString(KeyColumnTypes));

if (!Settings->HasShardIdHint()) {
state.IsFake = true;
InFlightShards.PushBack(&state);
ResolveShard(&state);
} else {
Expand Down Expand Up @@ -679,7 +681,9 @@ class TKqpReadActor : public TActorBootstrapped<TKqpReadActor>, public NYql::NDq
}

YQL_ENSURE(!newShards.empty());
Counters->IteratorsReadSplits->Add(newShards.size() - 1);
if (!state->IsFake) {
Counters->IteratorsReadSplits->Add(newShards.size() - 1);
}
if (Settings->GetReverse()) {
for (size_t i = 0; i < newShards.size(); ++i) {
PendingShards.PushBack(newShards[i].Release());
Expand Down

0 comments on commit bd81a67

Please sign in to comment.