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.

Sign up for GitHub

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

Jump to bottom

fix/QB-1344 Read IAVL cache size from config #193

Merged
merged 1 commit into from
Dec 22, 2022
Merged

fix/QB-1344 Read IAVL cache size from config #193

merged 1 commit into from
Dec 22, 2022

Conversation

mdumais-stratos
Copy link
Collaborator

https://qsn.atlassian.net/browse/QB-1344

  • Make the BaseApp actually use the IAVL cache size from the config (instead of always using the IAVL default of 500000)
  • To reduce memory usage, change the iavl-cache-size in app.toml to a more reasonable value. The default config value is 781250, which keeps way too many nodes in memory. Maybe 10000 would be more appropriate

@alexstratos alexstratos changed the base branch from dev to release December 22, 2022 17:07
@alexstratos alexstratos changed the base branch from release to dev December 22, 2022 17:07
@alexstratos alexstratos merged commit a99f6a9 into dev Dec 22, 2022
@alexstratos alexstratos deleted the fix/QB-1344_iavl_cache_size branch December 22, 2022 17:07
@mdumais-stratos mdumais-stratos mentioned this pull request Dec 22, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexstratos alexstratos alexstratos approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mdumais-stratos @alexstratos