Add memoization for MonadBlockchainCardanoNodeT queries related to the MonadBlockchain instance #228
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.
Not ready for merge!
Adding a
MemoizedCardanoNodeStateQueryT
monad transformer, which stores in-memory the responses of the different queries provided byMonadBlockchain
. Using the local node's state-query mini protocol is an inefficient way to query information (slow queries, especially if it is spammed with those queries + those queries slow down the other mini-protocols, such as the chain sync mini-protocol).In its current form, it is a proof of concept to showcase the idea, and see if this is worth adding. Even better if the reviewer has a different viewpoint on how to implement this.