amoeba opened a new pull request, #602:
URL: https://github.com/apache/arrow-go/pull/602

   ### Rationale for this change
   
   Prior to this PR, `TestMul_64bit`  in `internal/utils/math_64bit_test.go` 
fails on macOS aarch64. This wasn't caught on CI because there's no CI job for 
macOS aarch64. The behavior under test is technically undefined behavior 
(`int(NaN)` is UB).
   
   ### What changes are included in this PR?
   
   Just to keep the change small, I did the minimum necessary to make the 
assertion pass on all platforms.
   
   ### Are these changes tested?
   
   Yes.
   
   ### Are there any user-facing changes?
   
   No.
   
   Closes https://github.com/apache/arrow-go/issues/413
   


-- 
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]

Reply via email to