switch rust wrapper for sqlite in chia-tools #331
Merged
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.
rusqlite
appears to be much more established and popular in the rust ecosystem. But more importantly, it has much more complete support for sqlite features, which are required for the event-log analysis tool I'm building.While working in these files, I took the opportunity to transition
test_block_generators.rs
to use the (somewhat new) higher level abstraction to iterate over the transaction blocks in a blockchain database. This reduces a fair amount of duplicated code.