From a1740cd2e799807922d50f60b71d96aa6e3a1f09 Mon Sep 17 00:00:00 2001 From: Lukasz Mierzwa Date: Wed, 28 Aug 2024 10:47:40 +0100 Subject: [PATCH] Remove unnecessary locks Compact() is an uppercase function that deals with locks on its own, so we shouldn't have a lock around it. Signed-off-by: Lukasz Mierzwa --- tsdb/db.go | 2 -- 1 file changed, 2 deletions(-) diff --git a/tsdb/db.go b/tsdb/db.go index 001515081d..ccb0fa62bd 100644 --- a/tsdb/db.go +++ b/tsdb/db.go @@ -1518,9 +1518,7 @@ func (db *DB) compactBlocks() (err error) { default: } - db.mtx.RLock() uids, err := db.compactor.Compact(db.dir, plan, db.blocks) - db.mtx.RUnlock() if err != nil { return fmt.Errorf("compact %s: %w", plan, err) }