Skip to content
This repository was archived by the owner on Feb 23, 2026. It is now read-only.
This repository was archived by the owner on Feb 23, 2026. It is now read-only.

Memory leak in bidi resources #769

Closed

Description

@daniel-sanche

Reported in python-firestore: googleapis/python-firestore#985

It looks like some of the resources in BackgroundConsumer and ResumableBidiRpc are not being cleaned up properly on close, and preventing garbage collection. I ran the attached script, and can confirm I see the same thing

I opened #770 to address the issue, but maybe someone with more familiarity with the classes can take a look as well

Activity

  1. added
    type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.
    priority: p2Moderately-important priority. Fix may not be included in next release.
    on Dec 18, 2024
  2. vchudnov-g commented on Dec 18, 2024

    @vchudnov-g
    Contributor

    Thanks for reporting this and proposing a fix, @daniel-sanche! The "attached script" you refer to is the "Code example" in googleapis/python-firestore#985, correct?

  3. daniel-sanche commented on Dec 18, 2024

    @daniel-sanche
    ContributorAuthor

    Yes, that's the one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

priority: p2Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions