Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Rocksdb plugin to support OptimisticTransactionDb and TransactionDb #5328
Rocksdb plugin to support OptimisticTransactionDb and TransactionDb #5328
Changes from 8 commits
bf3b9f8
a6062c3
9b7d68e
de371a9
93322ea
46be43f
11c8a61
f8e848e
42e7924
c230472
c223ab5
c1c6bce
aaba2bc
4e24937
ebda4cf
3238f97
32adbf6
a0cdd20
298ab61
894dfb8
2076716
bc4e3fc
6ed4df6
80b9470
7cee988
df8a0aa
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
This could be done just for databaseVersion 1 in the switch statement as that indicates the db format is forest. Then the isForestStorageFormat wouldn't be needed either.
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.
I created that flag on purpose cause it took me some time to understand that DBversion actually is related to forest or bonsai. I think the usage of numbers to identify version is fine. But giving some more context to the code so next devs to touch don't spend time trying to understand what this version means.
In my first draft I wasted some time piping the StorageFormat through the plugin API because I didn't know that database version represented that same information.