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

Lower memory usage for aptly db cleanup #762

Merged
merged 1 commit into from
Jul 31, 2018
Merged

Commits on Jul 19, 2018

  1. Lower memory usage for aptly db cleanup

    This is not a complete fix, but the easiest first step.
    
    During `db cleanup`, aptly is loading every repo/mirror/... into memory,
    and even though each object is processed only once, collection holds
    a reference to all the loaded objects, so they won't be GC'd until
    process exits.
    
    CollectionFactory.Flush() releases pointers to collection objects,
    making objects egligble for GC.
    
    This is not a complete fix, as during iteration we could have tried
    to release a link to every object being GCed and that would have
    helped much more.
    smira committed Jul 19, 2018
    Configuration menu
    Copy the full SHA
    021b8c4 View commit details
    Browse the repository at this point in the history