Skip to content

Conversation

@vweevers
Copy link
Member

@vweevers vweevers commented Feb 2, 2025

@vweevers vweevers added the semver-minor New features that are backward compatible label Feb 2, 2025
Category: addition
@vweevers vweevers marked this pull request as ready for review February 2, 2025 22:28
@vweevers vweevers mentioned this pull request Feb 2, 2025
6 tasks
vweevers added a commit to Level/classic-level that referenced this pull request Apr 12, 2025
Also optimizes `db.get()` to avoid a copy of the key if it's a buffer.

Depends on Level/abstract-level#114.

Ref: Level/community#144
Category: addition
vweevers added a commit to Level/classic-level that referenced this pull request Apr 12, 2025
Also optimizes `db.get()` to avoid a copy of the key if it's a buffer.
Relies on a patch to LevelDB that replaces use of `std::string` with a
new ValueSink struct that gives us more control over the memory.

Depends on Level/abstract-level#114.

Ref: Level/community#144
Category: addition
@vweevers vweevers added the free-of-ai No generative AI was used for this issue or pull request label Apr 12, 2025
@vweevers vweevers merged commit 30f887d into main Apr 12, 2025
12 of 15 checks passed
@vweevers vweevers deleted the get-sync branch April 12, 2025 16:44
@github-project-automation github-project-automation bot moved this from Review to Done in Level Apr 12, 2025
vweevers added a commit to Level/classic-level that referenced this pull request Apr 12, 2025
Also optimizes `db.get()` to avoid a copy of the key if it's a buffer.
Relies on a patch to LevelDB that replaces use of `std::string` with a
new ValueSink struct that gives us more control over the memory.

Depends on Level/abstract-level#114.

Ref: Level/community#144
Category: addition
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

free-of-ai No generative AI was used for this issue or pull request semver-minor New features that are backward compatible

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants