Skip to content
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

Cleanup cache configuration code #7638

Closed
Tracked by #7634
jakmeier opened this issue Sep 19, 2022 · 1 comment
Closed
Tracked by #7634

Cleanup cache configuration code #7638

jakmeier opened this issue Sep 19, 2022 · 1 comment
Assignees
Labels
A-storage Area: storage and databases

Comments

@jakmeier
Copy link
Contributor

jakmeier commented Sep 19, 2022

Cache configuration is not ideal today. The format for cache capacities is verbose JSON, some field values are hard-coded without good reason, and the config code probably has too many structs (StoreConfig, TrieConfig, ShardCacheConfig).

We should carefully consider how the user-facing configuration should look like in the next step. And try to generally cleanup the code a bit more.

Already done: #7566

Already in progress: #7578

@jakmeier
Copy link
Contributor Author

Done with #7578

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-storage Area: storage and databases
Projects
None yet
Development

No branches or pull requests

1 participant