jackylee-ch opened a new pull request, #9841: URL: https://github.com/apache/paimon/pull/9841
### Purpose The fork swapped upstream's `Cleaner` for an explicit `close()` (`paimon-vortex-jni/README.md`), so two expressions now leak unreachable: `column(name)` drops the `root()` it passes to `getItem`, and `VortexRecordsReader:95` drops the inline `root()` plus its projection. Both are safe to release: vortex-jni builders do not take ownership (`expression.rs` module doc @ 0.73.0) and `scan.rs:82-92` clones the projection, so both are dead once `dataSource.scan` returns. ~96 bytes per predicate leaf: over 500k `column()` calls RSS grew 52896 KB before, 5984 KB after. ### Tests No regression test is possible — native allocation has no Java-visible counter. The 72 tests in `paimon-vortex-format` prove placement: every predicate leaf and read goes through both changed paths. The converter's operands and the pushed filter still leak; that needs a lifetime contract. Written with Claude Code; verification is mine. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
