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

Memory tuning on header synchronization #3940

Closed
chenyukang opened this issue Apr 17, 2023 · 0 comments · Fixed by #3970
Closed

Memory tuning on header synchronization #3940

chenyukang opened this issue Apr 17, 2023 · 0 comments · Fixed by #3970
Labels
t:bug Type: This doesn't seem right.

Comments

@chenyukang
Copy link
Collaborator

chenyukang commented Apr 17, 2023

With the merge of the previous PR: #3391

We may get some memory issue because of spacejam/sled#1388

As discussed with @zhangsoledad , a possible solution is:

  1. Replace sled. For this scenario, a btree database such as lmdb is more suitable. We are currently considering lmdb as a candidate, we need more tests and research.

  2. Add the original small memory mode back as a configurable option.

  3. Provide multiple configuration options to adapt to different scenarios, such as:

  • Stable operation under low memory conditions.
  • Fastest synchronization but with high memory usage.
  • Balanced performance.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t:bug Type: This doesn't seem right.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant