Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[wip]bt index: read value if key matches straight away #12786

Open
wants to merge 45 commits into
base: main
Choose a base branch
from
Open

Conversation

awskii
Copy link
Member

@awskii awskii commented Nov 19, 2024

another try to improve bt indices.

  • keep cursors in pool
  • reduce allocations to zero during seek
  • read Value immediately if key matches during Get (skip another ef.Get and decoding key twice)

Reduces load on GC and slightly improves performance

tried to use MatchCmp as well but this causes gas mismatch (i assume during reading from compressed file, because local tests are green though).

@awskii awskii added this to the 3.0.0-alpha6 milestone Nov 19, 2024
@awskii awskii changed the title bt index: read value if key matches straight away [wip]bt index: read value if key matches straight away Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant