Add WalletDb Initialization to MempoolNode::new() in src/mempool.rs #100
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.
PR Description: Add WalletDb Initialization to MempoolNode
Summary of Changes
This pull request introduces WalletDb initialization within the
MempoolNode::new()function located insrc/mempool.rs. The implementation mirrors the WalletDb setup found insrc/miner.rs, creating an instance of WalletDb during the initialization of MempoolNode.Motivation
The addition of WalletDb to MempoolNode is aimed at enhancing the functionality and consistency across components that depend on wallet operations. By ensuring that MempoolNode can access WalletDb, we improve its overall integration and enable future features that may require wallet interactions.
Background
The WalletDb component provides essential wallet functionalities that are already utilized in the miner context. To provide a similar level of access to the mempool, it's necessary to align MempoolNode with the existing architecture while adhering to the project's design principles.
Testing Instructions
To validate the implementation:
Known Issues / Limitations
Please review the changes and provide any feedback or suggestions. Thank you!