Skip to content

feat: expose enable multi writes#8

Merged
adsharma merged 3 commits intoLadybugDB:mainfrom
luca-iachini:feat-expose-enable-multi-writes
Apr 18, 2026
Merged

feat: expose enable multi writes#8
adsharma merged 3 commits intoLadybugDB:mainfrom
luca-iachini:feat-expose-enable-multi-writes

Conversation

@luca-iachini
Copy link
Copy Markdown
Contributor

This PR exposes the enableMultiWrites config.

@adsharma
Copy link
Copy Markdown
Contributor

@luca-iachini the PR looks good. Are you actually seeing any benefits after enabling multi writes though?

@luca-iachini
Copy link
Copy Markdown
Contributor Author

@luca-iachini the PR looks good. Are you actually seeing any benefits after enabling multi writes though?

Hey @adsharma, I opened this PR to align the Rust library with the database config. I’m testing it in a web server where multiple threads write to the same db instance.

@adsharma
Copy link
Copy Markdown
Contributor

I’m testing it in a web server where multiple threads write to the same db instance.

So you're doing it for functionality, not performance? The write throughput is actually lower if you have multiple writers going through the database engine. The p99 latency could be better though.

Wanted to make sure you're aware.

@adsharma adsharma merged commit 070470e into LadybugDB:main Apr 18, 2026
1 check passed
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.

2 participants