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 <[email protected]>
[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

Reply via email to