[#1691] metabase: Move cheaper conditions to the front in ListWithCursor()

`objectLocked` call is expensive, it does IO. We may omit it if object is not expired. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
2025-03-20 15:28:14 +03:00 · 2025-03-20 15:28:14 +03:00 · 8a5bff3587
commit 8a5bff3587
parent a49f0717b3
1 changed files with 1 additions and 1 deletions
--- a/pkg/local_object_storage/metabase/list.go
+++ b/pkg/local_object_storage/metabase/list.go
@ -251,7 +251,7 @@ func selectNFromBucket(bkt *bbolt.Bucket, // main bucket
 		}

 		expEpoch, hasExpEpoch := hasExpirationEpoch(&o)
-		if !objectLocked(bkt.Tx(), cnt, obj) && hasExpEpoch && expEpoch < currEpoch {
+		if hasExpEpoch && expEpoch < currEpoch && !objectLocked(bkt.Tx(), cnt, obj) {
 			continue
 		}