You are viewing a plain text version of this content. The canonical link for it is here.
Posted to builds@fluss.apache.org by GitBox <gi...@apache.org> on 2026/07/04 05:14:00 UTC
[GH] (fluss/fix/lance-committer-resource-leak): Workflow run "Check License" failed!
The GitHub Actions job "Check License" on fluss.git/fix/lance-committer-resource-leak has failed.
Run started by GitHub user XuQianJin-Stars (triggered by XuQianJin-Stars).
Head commit for run:
9ad17c15926d52049b49d0883d9eb05849387b76 / forwardxu <fo...@apache.org>
[lake/lance] Harden LanceLakeCommitter close() and commit() error paths
Two failure-side improvements to LanceLakeCommitter:
1. commit() now catches RuntimeException from LanceDatasetAdapter and
rethrows it as an IOException with the original cause preserved and
the dataset URI in the message. Previously, Lance JNI/runtime
exceptions escaped raw, so the tiering orchestrator could not treat
them uniformly with other IO failures and operators had no context
about which dataset failed.
2. close() now catches IllegalStateException that Arrow's RootAllocator
raises when child buffers were leaked, and rethrows it as an
IOException with an actionable message identifying the source. The
allocator is still closed on happy paths and its close() call is no
longer skipped by upstream failures - the exact leak text is kept as
the cause so that debugging remains straightforward.
Add LanceLakeCommitterTest covering:
- close() after construction with no work performed does not leak.
- commit() against a non-existent dataset raises IOException with a
RuntimeException cause.
- getMissingLakeSnapshot() against a missing dataset can be called and
the committer can still be closed cleanly afterwards.
Report URL: https://github.com/apache/fluss/actions/runs/28695824582
With regards,
GitHub Actions via GitBox