lriggs commented on code in PR #48867:
URL: https://github.com/apache/arrow/pull/48867#discussion_r2729067318
##########
cpp/src/gandiva/precompiled/time.cc:
##########
@@ -566,6 +567,28 @@ bool is_valid_time(const int hours, const int minutes,
const int seconds) {
seconds < 60;
}
+// Normalize sub-seconds value to milliseconds precision (3 digits).
+// Truncates if more than 3 digits are provided, pads with zeros if fewer than
3 digits
+ARROW_FORCE_INLINE
Review Comment:
I see that 'FORCE_INLINE' is used a lot, so maybe that is ok.
--
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]
