Skip to content

Fix memory leak in arrow export using array structure - #8953

Closed
wiredfool wants to merge 2 commits into
python-pillow:mainfrom
wiredfool:8950_arrow_leak
Closed

Fix memory leak in arrow export using array structure#8953
wiredfool wants to merge 2 commits into
python-pillow:mainfrom
wiredfool:8950_arrow_leak

Conversation

@wiredfool

@wiredfool wiredfool commented May 12, 2025

Copy link
Copy Markdown
Member

Fixes #8950 .

Changes proposed in this pull request:

  • Properly handle releasing the image associated with PyArrow capsules that have child arrays.

@wiredfool wiredfool mentioned this pull request May 13, 2025
Merged
1 task done
@wiredfool wiredfool added Bug Any unexpected behavior, until confirmed feature. Memory labels May 13, 2025
@radarhere

radarhere commented May 24, 2025

Copy link
Copy Markdown
Member

For awareness when reviewing - this is also included in #8954

@wiredfool

wiredfool commented May 30, 2025

Copy link
Copy Markdown
Member Author

Closing in favor of #8954, as that one actually has tests.

@wiredfool wiredfool closed this May 30, 2025
@aclark4life aclark4life added the Arrow Apache Arrow support (fromarrow, __arrow_c_array__) label Oct 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Arrow Apache Arrow support (fromarrow, __arrow_c_array__) Bug Any unexpected behavior, until confirmed feature. Memory

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Memory leak when calling PIL.Image.Image.__arrow_c_array__()

3 participants

@wiredfool @radarhere @aclark4life