summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSergey Belyakov <[email protected]>2026-07-06 18:13:19 +0300
committerGitHub <[email protected]>2026-07-06 15:13:19 +0000
commit4f5596bfa8940e24badded0bb2b43f49b047ec83 (patch)
tree5e6ee1ac6c98f0fa9f2289387477bcda189b9783
parent573ade75707844c6f3265679c5519b1f73e6a9fe (diff)
Fix false-positive VERIFY on key deletion in testshard (#45671)
-rw-r--r--ydb/core/test_tablet/load_actor_impl.cpp5
1 files changed, 4 insertions, 1 deletions
diff --git a/ydb/core/test_tablet/load_actor_impl.cpp b/ydb/core/test_tablet/load_actor_impl.cpp
index 380b678fe44..9baa4c5120b 100644
--- a/ydb/core/test_tablet/load_actor_impl.cpp
+++ b/ydb/core/test_tablet/load_actor_impl.cpp
@@ -42,7 +42,10 @@ namespace NKikimr::NTestShard {
}
return TString(sb);
};
- Y_VERIFY_S(info.ConfirmedState == ::NTestShard::TStateServer::CONFIRMED
+ // every pending transition from CONFIRMED state immediately sets ConfirmedKeyIndex -> Max<size_t>()
+ const bool shouldBeConfirmed = info.ConfirmedState == ::NTestShard::TStateServer::CONFIRMED
+ && info.PendingState == ::NTestShard::TStateServer::CONFIRMED;
+ Y_VERIFY_S(shouldBeConfirmed
? info.ConfirmedKeyIndex < ConfirmedKeys.size() && ConfirmedKeys[info.ConfirmedKeyIndex] == key
: info.ConfirmedKeyIndex == Max<size_t>(), makeError());
info.ConfirmedKeyIndex = Max<size_t>();