Fix bug with GetShardsCount() == 0 (ydb-platform#14812)

dahbka-lis · web-flow · commit dbf26c5a6032 · 2025-02-20T11:37:50.000+03:00
diff --git a/ydb/core/kqp/common/kqp_tx_manager.cpp b/ydb/core/kqp/common/kqp_tx_manager.cpp
@@ -280,7 +280,7 @@ class TKqpTransactionManager : public IKqpTransactionManager {
     }
 
     bool NeedCommit() const override {
-        const bool dontNeedCommit = IsReadOnly() && (IsSingleShard() || HasSnapshot());
+        const bool dontNeedCommit = IsEmpty() || IsReadOnly() && (IsSingleShard() || HasSnapshot());
         return !dontNeedCommit;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -280,7 +280,7 @@ class TKqpTransactionManager : public IKqpTransactionManager {`
`280`	`280`	`}`
`281`	`281`
`282`	`282`	`bool NeedCommit() const override {`
`283`		`- const bool dontNeedCommit = IsReadOnly() && (IsSingleShard() \|\| HasSnapshot());`
	`283`	`+ const bool dontNeedCommit = IsEmpty() \|\| IsReadOnly() && (IsSingleShard() \|\| HasSnapshot());`
`284`	`284`	`return !dontNeedCommit;`
`285`	`285`	`}`
`286`	`286`