fallintoplace opened a new pull request, #1586:
URL: https://github.com/apache/iceberg-go/pull/1586

   ## What changed
   
   Use signed floor division when converting nanosecond timestamps to 
microseconds. Add boundary coverage around zero and verify casts to both 
timestamp variants.
   
   ## Why
   
   Go integer division truncates toward zero. For pre-epoch timestamps with 
sub-microsecond precision, that could move the converted value forward by one 
microsecond and even cross the Unix epoch.
   
   ## Testing
   
   - `go test .`


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to