Skip to content

Conversation

mroeschke
Copy link
Member

@mroeschke mroeschke added the Performance Memory or execution speed performance label Jan 26, 2024
@mroeschke mroeschke added this to the 3.0 milestone Jan 26, 2024
@mroeschke mroeschke merged commit 8288b9b into pandas-dev:main Jan 26, 2024
@mroeschke mroeschke deleted the 54628-FIX-find-stack-level-memory-leak branch January 26, 2024 19:59
@mroeschke
Copy link
Member Author

Thanks @lmmx for the initial find

@jorisvandenbossche
Copy link
Member

Should we backport this for 2.2.x? (it's a memory leak, although presumably only a small one)

@mroeschke
Copy link
Member Author

Sure no harm in doing so

@mroeschke mroeschke modified the milestones: 3.0, 2.2.1 Jan 27, 2024
@mroeschke
Copy link
Member Author

@meeseeksdev backport 2.2.x

@mroeschke
Copy link
Member Author

@meeseeksmachine backport 2.2.x

meeseeksmachine pushed a commit to meeseeksmachine/pandas that referenced this pull request Jan 27, 2024
mroeschke added a commit that referenced this pull request Jan 27, 2024
… leak) (#57105)

Backport PR #57078: 54628 fix find stack level memory leak

Co-authored-by: Matthew Roeschke <[email protected]>
pmhatre1 pushed a commit to pmhatre1/pandas-pmhatre1 that referenced this pull request May 7, 2024
* fix: prevent potential memory leak

* test: unit test to cover find_stack_level utility function

* style: linting fixes and forgot to rename a funcdef

* style: linter fix and forgot to rename a variable in test

* test: simplify test and correct value (direct call stack level is 1 not 0)

* Add pydoc reference, remove file

* Call finally closer to its use

---------

Co-authored-by: Louis Maddox <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Performance Memory or execution speed performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BUG: memory leak in find_stack_level

4 participants