fallintoplace opened a new pull request, #1008: URL: https://github.com/apache/arrow-go/pull/1008
### Rationale for this change Timestamp coercion to a finer unit multiplies int64 values without checking whether the result fits. Values near the int64 limits can therefore wrap and be written as a different timestamp. ### What changes are included in this PR? - Check the int64 range before multiplying a valid timestamp. - Return `arrow.ErrInvalid` when coercion would overflow. - Ignore the stored value in null slots, matching the existing truncation checks. ### Are these changes tested? Yes. Focused tests cover positive and negative overflow as well as an overflowing value in a null slot. -- 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]
