-
Notifications
You must be signed in to change notification settings - Fork 222
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
add to kvdb-rocksdb create_if_missing config option #576
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you also put #[non_exhaustive]
above DatabaseConfig
and update the changelog?
https://rust-lang.github.io/rfcs/2008-non-exhaustive.html#structs-1
Thanks for the link @ordian I didn't know about this. However this will prevent downstream users from constructing or exhaustively matching the struct. Is it the intended behaviour? |
They can construct it with let mut config = DatabaseConfig::default();
config.create_if_missing = false; and exhaustive matching is disallowed. The intention is to allow adding more fields w/o making a breaking change. |
Published |
will be used to implement "auto backend" paritytech/substrate#9201
changes:
#[non_exhaustive]
above the DatabaseConfig