akravchukdremio commented on code in PR #48867:
URL: https://github.com/apache/arrow/pull/48867#discussion_r2733096769
##########
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:
@kou Got it, thanks for explanation. I've removed `ARROW_FORCE_INLINE` since
we don't need force inline
--
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]
