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

Increase default trie cache size to 1GiB #6546

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented Nov 19, 2024

The default trie cache size before was set to 64MiB, which is quite low to achieve real speed ups. 1GiB should be a reasonable number as the requirements for validators/collators/full nodes are much higher when it comes to minimum memory requirements. Also the cache will not use 1GiB from the start and fills over time. The setting can be changed by setting --trie-cache-size BYTE_SIZE.

The default trie cache size before was set to `64MiB`, which is quite low to achieve real speed ups.
`1GiB` should be a reasonable number as the requirements for validators/collators/full nodes are much
higher when it comes to minimum memory requirements. Also the cache will not use `1GiB` from the start
and fills over time. The setting can be changed by setting `--trie-cache-size BYTE_SIZE`.
@bkchr bkchr added the T0-node This PR/Issue is related to the topic “node”. label Nov 19, 2024
@bkchr bkchr requested a review from a team November 19, 2024 20:43
@bkchr
Copy link
Member Author

bkchr commented Nov 19, 2024

/cmd prdoc --bump patch --audience node-operator

Copy link

Command "prdoc --bump patch --audience node-operator" has failed ❌! See logs here

@bkchr
Copy link
Member Author

bkchr commented Nov 19, 2024

/cmd prdoc --bump patch --audience node_operator

\ low to achieve real speed ups. `1GiB` should be a reasonable number as the requirements\
\ for validators/collators/full nodes are much higher when it comes to minimum\
\ memory requirements. Also the cache will not use `1GiB` from the start and fills\
\ over time. The setting can be changed by setting `--trie-cache-size BYTE_SIZE`.\r\
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: removal of state_cache_size could be mentioned.

\n"
crates:
- name: sc-cli
bump: patch
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't removal of state_cache_size require a major here ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T0-node This PR/Issue is related to the topic “node”.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants