mirror of
https://github.com/seaweedfs/seaweedfs.git
synced 2026-10-22 06:22:38 +00:00
Replaces the PinRandomMode size heuristic with a per-chunk coverage rule. ViewFromVisibleIntervals already clips chunk views to the request window, so a view that is not IsFullChunk() is one the request only partially needs; fetch it as a range regardless of the detected read pattern. This closes the holes a request-size pin left open: ranges larger than SeqTolerance no longer revert to whole-chunk downloads once their buffers look sequential, and ranges that fully cover a chunk keep the shared whole-chunk path instead of fetching 256KiB slices piecemeal. Prefetch (MaybeCache) skips clipped views so it cannot amplify a range read either. PinRandomMode is dropped: no caller needs it once coverage drives the fetch choice. Range fetches route through fetchChunkDataFn so tests observe them the same way as whole-chunk downloads.
125 lines
4.6 KiB
Go
125 lines
4.6 KiB
Go
package filer
|
|
|
|
import (
|
|
"sync/atomic"
|
|
"testing"
|
|
)
|
|
|
|
const mb = 1 << 20
|
|
|
|
func TestReaderPatternSequentialAndHysteresis(t *testing.T) {
|
|
rp := NewReaderPattern()
|
|
rp.MonitorReadAt(0, mb) // near(0 vs frontier 0) -> +1
|
|
rp.MonitorReadAt(mb, mb) // +2
|
|
rp.MonitorReadAt(2*mb, mb) // +3 (capped)
|
|
if rp.IsRandomMode() {
|
|
t.Fatal("a stream from offset 0 must not be random")
|
|
}
|
|
// a single far outlier does not flip to random (hysteresis): +3 -> +2
|
|
rp.MonitorReadAt(900*mb, mb)
|
|
if rp.IsRandomMode() {
|
|
t.Fatal("a single outlier read must not flip to random mode")
|
|
}
|
|
}
|
|
|
|
func TestReaderPatternFarFirstReadIsRandom(t *testing.T) {
|
|
rp := NewReaderPattern()
|
|
rp.MonitorReadAt(500*mb, mb) // far from frontier 0 -> -1
|
|
if !rp.IsRandomMode() {
|
|
t.Fatal("a far first read should be random")
|
|
}
|
|
}
|
|
|
|
func TestReaderPatternToleranceAbsorbsReorder(t *testing.T) {
|
|
rp := NewReaderPattern()
|
|
rp.MonitorReadAt(0, mb) // +1, frontier 1MB
|
|
rp.MonitorReadAt(6*mb, mb) // |6-1|=5MB <= 8MB tolerance -> near, +2, frontier 7MB
|
|
rp.MonitorReadAt(3*mb, mb) // |3-7|=4MB <= 8MB -> near, +3
|
|
if rp.IsRandomMode() {
|
|
t.Fatal("reordered reads within tolerance must stay sequential")
|
|
}
|
|
}
|
|
|
|
func TestReaderPatternRandomRecoveryNeedsHysteresis(t *testing.T) {
|
|
rp := NewReaderPattern()
|
|
rp.MonitorReadAt(100*mb, mb) // far -> -1
|
|
rp.MonitorReadAt(300*mb, mb) // far -> -2
|
|
rp.MonitorReadAt(500*mb, mb) // far -> -3 (capped), frontier 501MB
|
|
// a single near read must not immediately flip back to sequential: -3 -> -2
|
|
rp.MonitorReadAt(501*mb, mb)
|
|
if !rp.IsRandomMode() {
|
|
t.Fatal("one near read must not flip back from deep random mode")
|
|
}
|
|
}
|
|
|
|
// The max-frontier (vs the old atomic.Swap of the last read's stop offset) is the
|
|
// load-bearing difference of this detector: the frontier must never move backward,
|
|
// or a backward read would lower the baseline and make later far reads look near.
|
|
func TestReaderPatternFrontierNeverRegresses(t *testing.T) {
|
|
rp := NewReaderPattern()
|
|
rp.MonitorReadAt(500*mb, mb) // far first read -> -1, frontier 501MB
|
|
rp.MonitorReadAt(0, mb) // a backward read must not pull the frontier back
|
|
if got := atomic.LoadInt64(&rp.readFrontier); got != 501*mb {
|
|
t.Fatalf("frontier regressed to %d; the max-frontier must never move backward", got)
|
|
}
|
|
// Behavioral consequence: reads far below the preserved frontier stay random.
|
|
// Had the frontier regressed to ~1MB, this would wrongly read as sequential.
|
|
if !rp.IsRandomMode() {
|
|
t.Fatal("reads far below the preserved frontier must remain random")
|
|
}
|
|
}
|
|
|
|
// SeqTolerance is the central new tuning knob; pin both sides of the inclusive
|
|
// boundary so an off-by-one or a '<' vs '<=' change can't slip through silently.
|
|
func TestReaderPatternToleranceBoundary(t *testing.T) {
|
|
atBoundary := NewReaderPattern()
|
|
atBoundary.MonitorReadAt(SeqTolerance, 0) // diff == SeqTolerance -> near (inclusive), +1
|
|
if atBoundary.IsRandomMode() {
|
|
t.Fatal("a read exactly at frontier+SeqTolerance must count as sequential")
|
|
}
|
|
pastBoundary := NewReaderPattern()
|
|
pastBoundary.MonitorReadAt(SeqTolerance+1, 0) // diff == SeqTolerance+1 -> far, -1
|
|
if !pastBoundary.IsRandomMode() {
|
|
t.Fatal("a read one byte past frontier+SeqTolerance must count as random")
|
|
}
|
|
}
|
|
|
|
// The escape from random mode: sustained near reads must climb the counter back
|
|
// out of the negative floor and re-enter sequential (whole-chunk cache) mode.
|
|
func TestReaderPatternRecoversFromRandom(t *testing.T) {
|
|
rp := NewReaderPattern()
|
|
rp.MonitorReadAt(100*mb, mb) // far -> -1
|
|
rp.MonitorReadAt(300*mb, mb) // far -> -2
|
|
rp.MonitorReadAt(500*mb, mb) // far -> -3 (capped), frontier 501MB
|
|
if !rp.IsRandomMode() {
|
|
t.Fatal("three far reads must be random")
|
|
}
|
|
// contiguous near reads: -3 -> -2 -> -1 -> 0 -> +1, back out of random mode
|
|
for i := int64(0); i < 4; i++ {
|
|
rp.MonitorReadAt(501*mb+i*mb, mb)
|
|
}
|
|
if rp.IsRandomMode() {
|
|
t.Fatal("sustained near reads must recover sequential mode")
|
|
}
|
|
}
|
|
|
|
// A ranged request's first read lands far from the frontier, but its
|
|
// remaining buffer reads are contiguous. The random verdict must stick for
|
|
// them — otherwise the tail of every range >256KiB pays a whole-chunk fetch.
|
|
func TestReaderPatternRangedReadStaysRandom(t *testing.T) {
|
|
rp := NewReaderPattern()
|
|
rp.MonitorReadAt(500*mb, 256*1024) // far first read -> -ModeChangeLimit
|
|
for i := 1; i <= 2; i++ {
|
|
rp.MonitorReadAt(500*mb+int64(i)*256*1024, 256*1024)
|
|
if !rp.IsRandomMode() {
|
|
t.Fatalf("contiguous read %d of a ranged request flipped back to sequential", i+1)
|
|
}
|
|
}
|
|
for i := 3; i < 10; i++ {
|
|
rp.MonitorReadAt(500*mb+int64(i)*256*1024, 256*1024)
|
|
}
|
|
if rp.IsRandomMode() {
|
|
t.Fatal("sustained sequential reads should restore sequential mode")
|
|
}
|
|
}
|