Skip to content

Commit bfc53e9

Browse files
Merge 4c4dc9f into 1d48d63
2 parents 1d48d63 + 4c4dc9f commit bfc53e9

File tree

2 files changed

+14
-4
lines changed

2 files changed

+14
-4
lines changed

ydb/core/tx/columnshard/engines/reader/actor/actor.cpp

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -153,8 +153,9 @@ void TColumnShardScan::HandleScan(TEvents::TEvWakeup::TPtr& /*ev*/) {
153153
<< " txId: " << TxId << " scanId: " << ScanId << " gen: " << ScanGen << " tablet: " << TabletId);
154154

155155
if (TMonotonic::Now() >= GetDeadline()) {
156-
SendScanError("ColumnShard scanner timeout: HAS_ACK=" + ::ToString(!!AckReceivedInstant));
157-
Finish(NColumnShard::TScanCounters::EStatusFinish::Deadline);
156+
CheckHanging(true);
157+
// SendScanError("ColumnShard scanner timeout: HAS_ACK=" + ::ToString(!!AckReceivedInstant));
158+
// Finish(NColumnShard::TScanCounters::EStatusFinish::Deadline);
158159
} else {
159160
ScheduleWakeup(GetDeadline());
160161
}
@@ -284,6 +285,14 @@ void TColumnShardScan::ContinueProcessing() {
284285
}
285286
}
286287
}
288+
CheckHanging();
289+
}
290+
291+
void TColumnShardScan::CheckHanging() const {
292+
AFL_WARN(NKikimrServices::TX_COLUMNSHARD)("HAS_ACK", AckReceivedInstant)("fi", FinishInstant)("si", !!ScanIterator)(
293+
"has_more", ChunksLimiter.HasMore())("in_waiting", ScanCountersPool.InWaiting())("counters_waiting", ScanCountersPool.DebugString())(
294+
"scan_actor_id", ScanActorId)("tx_id", TxId)("scan_id", ScanId)("gen", ScanGen)("tablet", TabletId)(
295+
"debug", ScanIterator ? ScanIterator->DebugString() : Default<TString>())("last", LastResultInstant);
287296
AFL_VERIFY(!!FinishInstant || !ScanIterator || !ChunksLimiter.HasMore() || ScanCountersPool.InWaiting())("scan_actor_id", ScanActorId)("tx_id", TxId)("scan_id", ScanId)(
288297
"gen", ScanGen)("tablet", TabletId)("debug", ScanIterator->DebugString())(
289298
"counters", ScanCountersPool.DebugString());

ydb/core/tx/columnshard/engines/reader/actor/actor.h

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,8 +47,8 @@ class TColumnShardScan: public TActorBootstrapped<TColumnShardScan>,
4747
private:
4848
STATEFN(StateScan) {
4949
auto g = Stats->MakeGuard("processing");
50-
TLogContextGuard gLogging(NActors::TLogContextBuilder::Build(NKikimrServices::TX_COLUMNSHARD_SCAN) ("SelfId", SelfId())(
51-
"TabletId", TabletId)("ScanId", ScanId)("TxId", TxId)("ScanGen", ScanGen)("task_identifier", ReadMetadataRange->GetScanIdentifier()));
50+
TLogContextGuard gLogging(NActors::TLogContextBuilder::Build(NKikimrServices::TX_COLUMNSHARD_SCAN) ("SelfId", SelfId())("TabletId",
51+
TabletId)("ScanId", ScanId)("TxId", TxId)("ScanGen", ScanGen)("task_identifier", ReadMetadataRange->GetScanIdentifier()));
5252
switch (ev->GetTypeRewrite()) {
5353
hFunc(NKqp::TEvKqpCompute::TEvScanDataAck, HandleScan);
5454
hFunc(NKqp::TEvKqp::TEvAbortExecution, HandleScan);
@@ -78,6 +78,7 @@ class TColumnShardScan: public TActorBootstrapped<TColumnShardScan>,
7878
void HandleScan(TEvents::TEvWakeup::TPtr& /*ev*/);
7979

8080
private:
81+
void CheckHanging(const bool logging = false) const;
8182
void MakeResult(size_t reserveRows = 0);
8283

8384
void AddRow(const TConstArrayRef<TCell>& row) override;

0 commit comments

Comments
 (0)