Skip to content

Commit 730917e

Browse files
committed
fix(kqp): always take snapshot for queries with stream lookup
1 parent af9b3a8 commit 730917e

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

ydb/core/kqp/common/kqp_tx.cpp

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -184,6 +184,11 @@ bool NeedSnapshot(const TKqpTransactionContext& txCtx, const NYql::TKikimrConfig
184184
return true;
185185
}
186186

187+
// We always need snapshot for stream lookup, besause it's used for dependent reads
188+
if (hasStreamLookup) {
189+
return true;
190+
}
191+
187192
// We don't want snapshot when there are effects at the moment,
188193
// because it hurts performance when there are multiple single-shard
189194
// reads and a single distributed commit. Taking snapshot costs

0 commit comments

Comments
 (0)