From 5646db1e3f657b71cac66003c33cb2a3268ae606 Mon Sep 17 00:00:00 2001 From: Alexander Rutkovsky Date: Wed, 10 Sep 2025 09:24:16 +0000 Subject: [PATCH] Correct FreeUpToLsn interpretation --- ydb/core/blobstorage/vdisk/common/blobstorage_dblogcutter.cpp | 2 +- .../blobstorage/vdisk/huge/blobstorage_hullhugerecovery.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/ydb/core/blobstorage/vdisk/common/blobstorage_dblogcutter.cpp b/ydb/core/blobstorage/vdisk/common/blobstorage_dblogcutter.cpp index 4e846396ef6f..4d9468ced0c5 100644 --- a/ydb/core/blobstorage/vdisk/common/blobstorage_dblogcutter.cpp +++ b/ydb/core/blobstorage/vdisk/common/blobstorage_dblogcutter.cpp @@ -119,7 +119,7 @@ namespace NKikimr { // only issue command if there is a progress in FreeUpToLsn queue bool progress = false; - for (; FreeUpToLsn && FreeUpToLsn.front() < curLsn; FreeUpToLsn.pop_front()) { + for (; FreeUpToLsn && FreeUpToLsn.front() <= curLsn; FreeUpToLsn.pop_front()) { progress = true; } diff --git a/ydb/core/blobstorage/vdisk/huge/blobstorage_hullhugerecovery.cpp b/ydb/core/blobstorage/vdisk/huge/blobstorage_hullhugerecovery.cpp index ea44d8eddfee..0946a3c7b4f6 100644 --- a/ydb/core/blobstorage/vdisk/huge/blobstorage_hullhugerecovery.cpp +++ b/ydb/core/blobstorage/vdisk/huge/blobstorage_hullhugerecovery.cpp @@ -349,7 +349,7 @@ namespace NKikimr { bool THullHugeKeeperPersState::WouldNewEntryPointAdvanceLog(ui64 freeUpToLsn, ui64 minInFlightLsn, ui32 itemsAfterCommit) const { - return freeUpToLsn < minInFlightLsn && (PersistentLsn <= freeUpToLsn || itemsAfterCommit > 10000); + return freeUpToLsn <= minInFlightLsn && (!PersistentLsn || PersistentLsn < freeUpToLsn || itemsAfterCommit > 10000); } // initiate commit