rustyconover commented on PR #11374: URL: https://github.com/apache/arrow-rs/pull/11374#issuecomment-6642051879
@Phoenix500526 thanks for identifying the schema-only stream case and fixing it in #11967. I've rebased this PR onto `23314565b`, which includes that fix. [e90ef77c5](https://github.com/apache/arrow-rs/pull/11374/commits/e90ef77c518e04a07bd91809a4ec9d03b1ba4aea) also adds wrapper-level coverage for schema-only streams with and without EOS, and empty record batches without EOS. The review follow-ups are pushed in [e90ef77c5](https://github.com/apache/arrow-rs/pull/11374/commits/e90ef77c518e04a07bd91809a4ec9d03b1ba4aea) and [a844fcf7d](https://github.com/apache/arrow-rs/pull/11374/commits/a844fcf7d478df5db0cf0e3a3b1dd97f27e00190), with replies on the individual threads. Validation on EC2 passed: **7,616 workspace tests**, zero failures, 30 ignored tests; the all-feature IPC suite and workspace Clippy with all targets/features also passed. Formatting and spelling checks passed locally, and all 29 selected benchmark cases completed. The latest [fork CI run](https://github.com/rustyconover/arrow-rs/actions/runs/41908711226) has also passed. I've left the review threads open for confirmation. -- 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]
