Skip to content

This hacky fix shows an improvement in memory usage for iceberg#13297 - #13298

Closed
jkolash wants to merge 2 commits into
apache:mainfrom
jkolash:hacky-fix-for-13297
Closed

This hacky fix shows an improvement in memory usage for iceberg#13297#13298
jkolash wants to merge 2 commits into
apache:mainfrom
jkolash:hacky-fix-for-13297

Conversation

@jkolash

Copy link
Copy Markdown
Contributor

The program was able to run to completion, but there were still leaks.

image

@jkolashjkolash changed the title This hacky fix shows an improvement in memory ussage for iceberg#13297This hacky fix shows an improvement in memory usage for iceberg#13297Jun 11, 2025
@jkolash

Copy link
Copy Markdown
ContributorAuthor

Running with the additional set of changes makes it so heap never even reaches 8 gigs.

image

I ran this while doing zoom so it ran longer, but these changes dramatically reduce the amount of memory used for what I was doing.

@github-actions

Copy link
Copy Markdown

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions.

@jkolash

Copy link
Copy Markdown
ContributorAuthor

This is a bug in spark not in iceberg. This should be fixed in apache/spark#51503

@jkolashjkolash closed this Jul 17, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@jkolash