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 <lukasz@cloudflare.com>

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)
 		}