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

port bytes size iavl cache to v6.x to mitigate RAM issues #1239

Closed
p0mvn opened this issue Apr 12, 2022 · 2 comments
Closed

port bytes size iavl cache to v6.x to mitigate RAM issues #1239

p0mvn opened this issue Apr 12, 2022 · 2 comments
Labels
A:backport/v6.x backport patches to v6.x branch Good first issue

Comments

@p0mvn
Copy link
Member

p0mvn commented Apr 12, 2022 •
edited
Loading

Background

There are RAM issues on v6.x with rocks db. It is likely to be caused by iavl fast storage. Once this PR is merged, it should be backported to v6.x.

Originally posted by @p0mvn in #1140 (comment)

Acceptance Criteria

  • bytes size cache is backported to v6.x
@p0mvn p0mvn added the A:backport/v6.x backport patches to v6.x branch label Apr 12, 2022
@faddat
Copy link
Member

faddat commented Apr 14, 2022

sir, you have your ear close to the metal, and thank you.

I'd meant to open just this issue.

@faddat faddat mentioned this issue Apr 18, 2022
Closed
4 tasks
@ValarDragon
Copy link
Member

Closing for now, only affects archive nodes and there are better ideas to do now (e.g. IAVL v1)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A:backport/v6.x backport patches to v6.x branch Good first issue
Projects
Osmosis Chain Development
Archived in project
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ValarDragon @faddat @p0mvn