davidzollo commented on PR #10958: URL: https://github.com/apache/seatunnel/pull/10958#issuecomment-5391955074
Closing this PR as superseded by #11929, which republishes the same two functional commits on an Apache-owned branch (as this PR's fork-branch CI could not represent the current head reliably) and additionally addresses the review findings raised here by @SEZ9 and on #11929 by @DanielLeens: - unchecked exceptions from `ArrowStreamReader.close()` are now aggregated with allocator close failures via `addSuppressed`; - real-allocator round-trip regression tests were added (including a dictionary-encoded stream, which is the actual issue #9863 failure signature), replacing the mock-only coverage this PR shipped with. Please follow up on #11929. Note #11593 remains open as an independent fix for the same defect and still needs a maintainer decision on which implementation lands. -- 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]
