akravchukdremio opened a new pull request, #48867:
URL: https://github.com/apache/arrow/pull/48867

   ### Rationale for this change
   
   Fixes #48866. The Gandiva precompiled time functions `castTIMESTAMP_utf8` 
and `castTIME_utf8` currently reject timestamp and time string literals with 
more than 3 subsecond digits (beyond millisecond precision), throwing an 
"Invalid millis" error. This behavior is inconsistent with other 
implementations.
   
   ### What changes are included in this PR?
   
   - Fixed `castTIMESTAMP_utf8` and `castTIME_utf8` functions to truncate 
subseconds beyond 3 digits instead of throwing an error
   - Updated tests. Replaced error-expecting tests with truncation verification 
tests and added edge cases
   
   ### Are these changes tested?
   
   Yes
   
   ### Are there any user-facing changes?
   
   No


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