mrdrivingduck opened a new pull request, #182: URL: https://github.com/apache/paimon-cpp/pull/182
I found this while debugging an S3 crash. I initially suspected the S3 filesystem, but the issue was in the Arrow input-stream adapter: a returned buffer can outlive its memory pool when a reader is closed while a caller still holds a buffer. This is more likely with network filesystems, where asynchronous reads may finish after the reader is closed. This patch uses `shared_ptr`'s aliasing constructor to keep the pool alive with the returned buffer. It also adds deterministic coverage for `Read`, `ReadAt`, and `ReadAsync` using a small `InputStream` mock. Without the change, the tests crash; with it, they pass. -- 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]
