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

Fix memory leak #132

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix memory leak #132

wants to merge 1 commit into from

Commits on Oct 11, 2024

  1. Fix memory leak 

    Creating a new session each time S3Storage is instantiated creates a memory leak. It seems that S3Storage can get created a bunch of times (I'm seeing it get created again and again as my app runs) and a boto3's Session takes loads of memory (see boto/boto3#1670) so my app eventually runs out of memory. This should fix the issue while still avoiding using the same session across different threads.
    @almost
    almost committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    668ae88 View commit details
    Browse the repository at this point in the history