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

High RAM consumption #260

Open
samuel-rufi opened this issue Dec 7, 2021 · 9 comments
Open

High RAM consumption #260

samuel-rufi opened this issue Dec 7, 2021 · 9 comments
Labels
bug Something isn't working

Comments

@samuel-rufi
Copy link

samuel-rufi commented Dec 7, 2021

Hi @shrimalmadhur, I would like to give you a status update from my side:
The very high RAM consumption due to the "memory_limit_disabled" = true hotfix for #245 now is indeed a problem as rosetta-cli breaks down after some hours already with 16 GB RAM.
Unfortunately I also can't run the rosetta-cli without "memory_limit_disabled" = true else it will break in the account bootstrapping phase as it can't handle large bootsrapping.json files #245.

I guess it all comes back to the Badger Tx size / or how the Badger commits are done. Maybe this could be adjusted or are there other options?

Many thanks for your time and efforts.

@samuel-rufi samuel-rufi added the bug Something isn't working label Dec 7, 2021
@shrimalmadhur
Copy link
Contributor

Hey @rufsam, thanks for raising this. Yea it does come back to tx size. We might need to optimize how we do badger transactions. I have not gotten a chance to work on it yet but it is in my radar.

Also feel free to contribute if you have some cycles and can work on resolving this. We always welcome contributions.

@shiatcb
Copy link
Contributor

shiatcb commented Dec 7, 2021

Hi @rufsam , would you mind providing the bootstrap_balances.json with 35k accounts so we can repro this issue? Feel free to drop the file in your comments or email shi.cheng@coinbase.com, whatever you prefer.

@samuel-rufi
Copy link
Author

samuel-rufi commented Dec 7, 2021

Thanks alot @shrimalmadhur @shiatcb for your fast answers and help. This is the bootstrap_balances.zip I'm using. So after loading around 35k accounts the badger error should appear. If there is anything I can help you with, just tell me! 👍

@shiatcb
Copy link
Contributor

shiatcb commented Dec 8, 2021

@rufsam thanks for the quick reply! Also are you able to provide the config for peering.json (devnet/mainnet) with multiaddress? I am now following the deployment procedure and trying to bring up node svc & rosetta svr.

@samuel-rufi
Copy link
Author

samuel-rufi commented Dec 8, 2021

Hi @shiatcb, many thanks for working through it 👍 Maybe we should switch to e-mail so that I can provide you with better assistance. I will contact you with all the required info!

@shiatcb
Copy link
Contributor

shiatcb commented Dec 8, 2021

My pleasure, you can email shi.cheng@coinbase.com whenever you have the info.

@samuel-rufi
Copy link
Author

Sent! 😄

@muXxer
Copy link

muXxer commented Dec 8, 2021

Hey! Just wanted to mention that we had a lot of problems with BadgerDB in our own project for years.
After switching to RocksDB, memory usage dropped significantly, and speed also increased dramatically.

We implemented an Interface for our software called KVStore so we can switch the used database engine easily (gRocksDB, Badger, bbolt, pebbledb).

@shrimalmadhur
Copy link
Contributor

@muXxer that's a very interesting point. Are there any data points which I can refer on how much improvement was there? If this makes us more memory efficient, we could definitely consider moving to RocksDB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

No branches or pull requests

4 participants