pump/storage: Only hold the lock when collecting logFiles to scan

pump/storage/vlog.go

@@ -408,18 +408,20 @@ func (vlog *valueLog) scanRequests(start valuePointer, fn func(*request) error)
 
 // currently we only use this in NewAppend** to scan the record which not write to KV but in the value log, so it's OK to hold the vlog.filesLock lock
 func (vlog *valueLog) scan(start valuePointer, fn func(vp valuePointer, record *Record) error) error {
-	vlog.filesLock.Lock()
-	defer vlog.filesLock.Unlock()
-
+	vlog.filesLock.RLock()
 	fids := vlog.sortedFids()
-
+	var lfs []*logFile
 	for _, fid := range fids {
-		if fid < start.Fid {
-			continue
+		if fid >= start.Fid {
+			lf := vlog.filesMap[fid]
+			lfs = append(lfs, lf)
 		}
-		lf := vlog.filesMap[fid]
+	}
+	vlog.filesLock.RUnlock()
+
+	for _, lf := range lfs {
 		var startOffset int64
-		if fid == start.Fid {
+		if lf.fid == start.Fid {
 			startOffset = start.Offset
 		}
 		err := lf.scan(startOffset, fn)