fallintoplace opened a new pull request, #998: URL: https://github.com/apache/arrow-go/pull/998
### Rationale for this change Minimum compression savings is documented as a value in the range [0,1]. NaN bypassed both range comparisons and was silently accepted. ### What changes are included in this PR? Reject NaN alongside the existing out-of-range checks and include it in the existing validation test. ### Are these changes tested? Yes. go test ./arrow/ipc ### Are there any user-facing changes? IPC writers now return an invalid-argument error for a NaN minimum space savings value. -- 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]
