Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[tiering] Fix Paimon IOManager leak in MergeTreeWriter causing /tmp disk exhaustion#3193

Merged
luoyuxia merged 1 commit intoapache:release-0.9apache/fluss:release-0.9from
luoyuxia:release-0.9-fix-io-managerluoyuxia/fluss:release-0.9-fix-io-managerCopy head branch name to clipboard
Apr 24, 2026
Merged

[tiering] Fix Paimon IOManager leak in MergeTreeWriter causing /tmp disk exhaustion#3193
luoyuxia merged 1 commit intoapache:release-0.9apache/fluss:release-0.9from
luoyuxia:release-0.9-fix-io-managerluoyuxia/fluss:release-0.9-fix-io-managerCopy head branch name to clipboard

Conversation

@luoyuxia
Copy link
Copy Markdown
Contributor

The IOManager created in MergeTreeWriter.createTableWrite() was passed inline to withIOManager() without keeping a reference. Since Paimon's TableWriteImpl does not own the IOManager lifecycle, it was never closed, leaving paimon-io- directories in /tmp after each tiering task.

This fix stores the IOManager as a field and explicitly closes it in close() via a finally block, ensuring temp directories are cleaned up.

Purpose

Linked issue: close #xxx

Brief change log

Tests

API and Format

Documentation

…isk exhaustion

The IOManager created in MergeTreeWriter.createTableWrite() was passed
inline to withIOManager() without keeping a reference. Since Paimon's
TableWriteImpl does not own the IOManager lifecycle, it was never closed,
leaving paimon-io-<UUID> directories in /tmp after each tiering task.

This fix stores the IOManager as a field and explicitly closes it in
close() via a finally block, ensuring temp directories are cleaned up.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@luoyuxia luoyuxia merged commit 5c6f6b1 into apache:release-0.9 Apr 24, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.