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.

Sign up for GitHub

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

Jump to bottom

Create a standalone LevelDB batch size estimator #6290

Open
str4d opened this issue Dec 5, 2022 · 0 comments
Open

Create a standalone LevelDB batch size estimator #6290

str4d opened this issue Dec 5, 2022 · 0 comments
Labels
C-upstream-port Category: Changes that are ported from the Bitcoin Core codebase. M-upstreamable Includes changes that should probably be provided as a PR on Bitcoin Core
Milestone
Post 5.9.0

Comments

@str4d
Copy link
Contributor

str4d commented Dec 5, 2022

bitcoin/bitcoin@e66dbde (the first commit that #6286 would backport) adds the ability for a LevelDB write batch to estimate its own size. We want to be able to estimate the size of a potential batch before creating it. We should backport this upstream commit and then refactor it to extract a DBBatchSizeEstimator.
@str4d str4d added C-upstream-port Category: Changes that are ported from the Bitcoin Core codebase. M-upstreamable Includes changes that should probably be provided as a PR on Bitcoin Core labels Dec 5, 2022
@daira daira added this to the Release 5.4.0 milestone Dec 5, 2022
@daira daira mentioned this issue Dec 6, 2022
Closed
@nuttycom nuttycom modified the milestones: Release 5.4.0, Release 5.5.0 Jan 9, 2023
@nuttycom nuttycom modified the milestones: Release 5.5.0, Release 5.6.0 Jan 30, 2023
@nuttycom nuttycom removed this from the Release 5.6.0 milestone Mar 23, 2023
@nuttycom nuttycom added this to the Post 5.7.0 milestone Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-upstream-port Category: Changes that are ported from the Bitcoin Core codebase. M-upstreamable Includes changes that should probably be provided as a PR on Bitcoin Core
Projects
None yet
Milestone
Post 5.9.0
Development

No branches or pull requests
3 participants
@nuttycom @daira @str4d