Skip to content
This repository was archived by the owner on Jan 20, 2026. It is now read-only.

CacheKv optimizations #113

Merged
BrandonWeng merged 8 commits intomainfrom
bweng-debug-check-tx
Dec 15, 2022
Merged

CacheKv optimizations #113
BrandonWeng merged 8 commits intomainfrom
bweng-debug-check-tx

Conversation

@BrandonWeng
Copy link
Contributor

@BrandonWeng BrandonWeng commented Dec 14, 2022

Describe your changes and provide context

Cherry picking changes here:

  • #14168 perf: store/cachekv: preallocate kvL in dirtyItems which gets appended too
  • #10024 fix!: store/cachekv: reduce growth factor for iterator ranging using binary searches #10024

Testing performed to validate your change

Ran a cluster
image

if item.Value == nil {
// deleted element, tracked by store.deleted
// setting arbitrary value
// TODO: Don't ignore this error.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol

@BrandonWeng BrandonWeng merged commit a7c6045 into main Dec 15, 2022
@BrandonWeng BrandonWeng deleted the bweng-debug-check-tx branch December 15, 2022 01:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants