You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
we store the mempool in memory, so it doesn't make sense to have it as part of storage. Instead, is should be encapsulated inside the blockchain object.
We should probably wait for a few prs to go in to avoid merge conflicts, like: #1990
The text was updated successfully, but these errors were encountered:
**Motivation**
The `mempool` is currently stored in the `Store` struct, it makes more
sense to have it in the `Blockchain` struct
**Description**
- Move all the functions related to mempool to a new struct called
`Mempool` that implements all the functions related to the mempool.
- To add new transactions to the mempool we need to do some validations,
for this we need access to the store so some mempool functions are
exposed by the blockchain struct
- `Blockchain::add_transaction_to_pool`,
- `Blockchain::add_blob_transaction_to_pool`
- `Blockchain::remove_transaction_from_pool`
Closes#2072
we store the mempool in memory, so it doesn't make sense to have it as part of storage. Instead, is should be encapsulated inside the
blockchain
object.We should probably wait for a few prs to go in to avoid merge conflicts, like:
#1990
The text was updated successfully, but these errors were encountered: