-
Notifications
You must be signed in to change notification settings - Fork 33
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
Make sure to GC as much as possible #56
Conversation
I'm not noticing any difference when testing with go-ipfs. Badger still isn't GCing anyting when running |
I ran 81G 65G 63G 62G 60G 58G 58G 57G 57G 57G 57G 56G 56G 50G 47G 44G 44G 44G 41G 34G 34G 34G 34G 33G 33G 32G 32G 32G 32G 32G 31G 31G 30G 30G 30G 30G 30G 30G 30G 29G 29G 29G 29G 29G 29G 29G 29G 29G 29G 29G 29G 29G 29G 29G 29G 29G 29G 29G 29G 29G 29G For reference, running |
@BenLubar thanks for testing. Can you run
in both repos (or at least the GCed one)? I've also noticed that we are actually running https://github.com/dgraph-io/badger/tree/8115aed38f8f8cd248d832f51c27700efc25b201, which does not include the fix I was talking about on IRC (dgraph-io/badger#725). I've pushed an update to it now, so when you do @BenLubar Can you try running with that fix on GCed database? |
The badgerds backup/restore versions:
I'll update this with the version from this PR once it finishes. Edit: Here's what it looks like with the version from this PR:
|
@magik6k Here's the output of
|
cc @manishrjain |
Ideally we'd pass a context to this. IIRC badger GC is stop-the-world, so if it runs too long it can easily starve the rest of the system (insofar datastore access is concerned). It's kinda dangerous. |
Do you guys still need help with this? IIRC, some changes were made to Badger to improve GC. |
@magik6k can we close this? |
I think we still want this (but possibly with a context as raul pointed out) - without this we'll only gc at most 1gb - https://github.com/dgraph-io/badger#garbage-collection |
Changes in this PR & the suggestions in the comments have now been incorporated in #72. |
No description provided.