Skip to content

feat: default enable_multi_writes to true - #76

Closed
adsharma wants to merge 1 commit into
mainfrom
enable-multi-writes-default-true
Closed

adsharma wants to merge 1 commit into
mainfrom
enable-multi-writes-default-true

Conversation

@adsharma

Copy link
Copy Markdown
Contributor

Tracks LadybugDB/ladybug#1435, which renames debug_enable_multi_writes to enable_multi_writes and flips the engine default to true.

This binding passes its own default explicitly, so without this change Python users would silently stay on single-writer mode despite the new engine default. Mirrors the flip in database.py, _lbug_capi.py, and py_database.cpp.

Tracks LadybugDB/ladybug#1435, which renames debug_enable_multi_writes
to enable_multi_writes and flips the engine default to true. Mirror
that default here so the binding does not silently pin single-writer
mode.
@adsharma

Copy link
Copy Markdown
Contributor Author

Superseded by #77, which takes the fuller approach: all settings default to None (engine default) instead of just flipping this one value.

@adsharma adsharma closed this Oct 27, 2026
@adsharma
adsharma deleted the enable-multi-writes-default-true branch October 27, 2026 01:37
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