-
Notifications
You must be signed in to change notification settings - Fork 4.1k
split: replace seeded random source #94752
Copy link
Copy link
Closed
Labels
A-kv-distributionRelating to rebalancing and leasing.Relating to rebalancing and leasing.C-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)T-kvKV TeamKV Team
Description
In #93838 we started using a seeded rand for use in the load based +
splitter's reservoir sampling algorithm.
rand.Soure heap allocates on init approximately 4kb. When intialized
per-replica, this is problematic as nodes scale to large replica counts.
The splitter should instead be initialized with the global rand, to reduce
memory overhead per-replica.
xref #94737
Jira issue: CRDB-23121
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-kv-distributionRelating to rebalancing and leasing.Relating to rebalancing and leasing.C-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)T-kvKV TeamKV Team