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

Use redb instead of SQLite #1915

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Use redb instead of SQLite #1915

wants to merge 1 commit into from

Conversation

JamesHinshelwood
Copy link
Contributor

@JamesHinshelwood JamesHinshelwood commented Nov 28, 2024

The performance of redb in devnet compared to SQLite is much faster. I don't necessarily blame SQLite for this, since it could easily be our transaction management, SQL queries, lack of indices, etc. that are making the existing implementation slow. However, I do feel confident saying that this interface makes it clearer what work the database is doing and makes it harder to accidentally do slow things.

@shawn-zil
Copy link
Contributor

Can we run it for a week and measure the block-slowdown timings (if any)?

@JamesHinshelwood JamesHinshelwood force-pushed the redb branch 3 times, most recently from fa0e0b6 to 7c83587 Compare December 4, 2024 14:14
@JamesHinshelwood JamesHinshelwood force-pushed the redb branch 2 times, most recently from f36a60e to a5f4f48 Compare December 9, 2024 20:40
Copy link
Contributor

github-actions bot commented Dec 9, 2024

🐰 Bencher Report

Branchredb
Testbedself-hosted
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
nanoseconds (ns)
(Result Δ%)
Upper Boundary
nanoseconds (ns)
(Limit %)
process-empty/process-empty📈 view plot
🚷 view threshold
9,061,300.00
(-0.94%)
10,577,299.32
(85.67%)
produce-full/produce-full📈 view plot
🚷 view threshold
1,310,700,000.00
(-28.67%)
2,508,211,219.86
(52.26%)
🐰 View full continuous benchmarking report in Bencher

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.

Missing perfectly divisible deposits
2 participants