-
Notifications
You must be signed in to change notification settings - Fork 398
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
sdk/params: add lazy caching with LRU #3023
Comments
4 tasks
zivkovicmilos
pushed a commit
that referenced
this issue
Nov 12, 2024
- [x] add `r/sys/params` - [x] add `genesis/genesis_params.toml` - [x] port some existing configurations - [x] open issue: add LRU lazy caching with instant invalidation using a transient store (#3023) Depends on #2920 Depends on #3003 (cherry-picked) Blocking #2911 --------- Signed-off-by: moul <[email protected]> Co-authored-by: Morgan <[email protected]>
r3v4s
pushed a commit
to gnoswap-labs/gno
that referenced
this issue
Dec 10, 2024
- [x] add `r/sys/params` - [x] add `genesis/genesis_params.toml` - [x] port some existing configurations - [x] open issue: add LRU lazy caching with instant invalidation using a transient store (gnolang#3023) Depends on gnolang#2920 Depends on gnolang#3003 (cherry-picked) Blocking gnolang#2911 --------- Signed-off-by: moul <[email protected]> Co-authored-by: Morgan <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
sdk/params
will be used in various parts of the project to enable dynamic configuration of the chain, controlled by GovDAO.To avoid managing event-based logic and to ensure good performance, we should implement a lazy caching mechanism for
sdk/params
.This mechanism will allow each call to the param getters to first check the cache and return the value if it is available. If not, it will retrieve the value from the store and save it in the cache. We should also implement automatic cache invalidation at the end of the block when a variable changes.
Additionally, there should be an LRU system that allows a node operator to select a maximum cache size.
Related with #3003
The text was updated successfully, but these errors were encountered: