pchintar commented on PR #9778: URL: https://github.com/apache/arrow-rs/pull/9778#issuecomment-4314929065
Thanks ā So, now it seems to be faster for a majority of cases in `ipc_reader` but besides that, there's an integration error in the checks. So I checked the integration failure and found the root cause: The pure `Vec + read_to_end` version breaks IPC inter-operability in ``` union nanoarrow producing ā Rust consuming ``` Specifically: ``` arrow-stream-to-file ... returned non-zero exit status 101 ``` This indicates that reusing the `Vec` allocation directly is not safe for all IPC cases, likely due to alignment requirements (`arrow_buffer::buffer::ALIGNMENT`). Iām reverting this change for now and will revisit a safe approach that preserves alignment while avoiding any overhead. -- 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]
