yinli-systems commented on issue #10131: URL: https://github.com/apache/arrow-rs/issues/10131#issuecomment-5532903404
Thanks @edubraqd for the concrete reproducer. This exact `Time64(Microsecond) -> Time64(Nanosecond)` path is covered by #10907: the unchecked multiplication is replaced with `checked_mul` in safe mode and `mul_checked` in unsafe mode, with `i64::MIN`/`i64::MAX` regression coverage in `test_cast_time64_overflow`. I reran `cargo test -p arrow-cast test_cast_time64_overflow` on the current PR head (`0fbc616bc`): 1 passed, 0 failed. The full PR CI is also green (39 successful checks, 1 skipped), and the PR is waiting for re-review. This response was prepared with AI assistance. -- 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]
