Skip to content

Commit 51a56ba

Browse files
Apply suggestion from @StephenButtolph
Co-authored-by: Stephen Buttolph <stephen@avalabs.org> Signed-off-by: Sam Liokumovich <65994425+samliok@users.noreply.github.com>
1 parent fa6fb35 commit 51a56ba

File tree

1 file changed

+8
-10
lines changed

1 file changed

+8
-10
lines changed

database/leveldb/db.go

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -200,17 +200,15 @@ type config struct {
200200

201201
// randomizeCompactionParams sets compaction parameters in cfg to random values
202202
// to spread out compaction behavior across different nodes.
203+
//
204+
// #nosec G404 // non-crypto randomness is fine here
203205
func randomizeCompactionParams(cfg *config) {
204-
r := rand.New(rand.NewSource(time.Now().UnixNano())) // #nosec G404 -- non-crypto randomness is fine here
205-
206-
// generate random defaults for CompactionL0Trigger
207-
compactionL0Trigger := r.Intn(maxCompactionL0Trigger-minCompactionL0Trigger+1) + minCompactionL0Trigger
208-
209-
// generate random defaults for CompactionTableSize
210-
compactionTableSize := (r.Intn(maxCompactionTableSize-minCompactionTableSize+1) + minCompactionTableSize)
211-
212-
cfg.CompactionL0Trigger = compactionL0Trigger
213-
cfg.CompactionTableSize = compactionTableSize
206+
const (
207+
rangeCompactionL0Trigger = maxCompactionL0Trigger - minCompactionL0Trigger + 1
208+
rangeCompactionTableSize = maxCompactionTableSize - minCompactionTableSize + 1
209+
)
210+
cfg.CompactionL0Trigger = rand.Intn(rangeCompactionL0Trigger) + minCompactionL0Trigger
211+
cfg.CompactionTableSize = rand.Intn(rangeCompactionTableSize) + minCompactionTableSize
214212
}
215213

216214
// New returns a wrapped LevelDB object.

0 commit comments

Comments
 (0)