use the constant manifest.NumLevels instead of literal value

cockroachdb · Oct 17, 2023 · 5807b59 · 5807b59
1 parent 94ccf35
commit 5807b59
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/compaction_picker.go b/compaction_picker.go
@@ -1663,7 +1663,7 @@ func pickAutoLPositive(
 	vers *version,
 	cInfo candidateLevelInfo,
 	baseLevel int,
-	levelMaxBytes [7]int64,
+	levelMaxBytes [numLevels]int64,
 ) (pc *pickedCompaction) {
 	if cInfo.level == 0 {
 		panic("pebble: pickAutoLPositive called for L0")

diff --git a/tool/logs/compaction.go b/tool/logs/compaction.go
@@ -19,9 +19,12 @@ import (
 
 	"github.com/cockroachdb/errors"
 	"github.com/cockroachdb/pebble/internal/humanize"
+	"github.com/cockroachdb/pebble/internal/manifest"
 	"github.com/spf13/cobra"
 )
 
+const numLevels = manifest.NumLevels
+
 var (
 	// Captures a common logging prefix that can be used as the context for the
 	// surrounding information captured by other expressions. Example:
@@ -514,8 +517,8 @@ type windowSummary struct {
 	compactionBytesMoved map[fromTo]uint64
 	compactionBytesDel   map[fromTo]uint64
 	compactionTime       map[fromTo]time.Duration
-	ingestedCount        [7]int
-	ingestedBytes        [7]uint64
+	ingestedCount        [numLevels]int
+	ingestedBytes        [numLevels]uint64
 	readAmps             []readAmp
 	longRunning          []event
 }