jorisvandenbossche commented on code in PR #12657:
URL: https://github.com/apache/arrow/pull/12657#discussion_r869197050


##########
cpp/src/arrow/compute/api_scalar.h:
##########
@@ -117,6 +119,18 @@ class ARROW_EXPORT RoundTemporalOptions : public 
FunctionOptions {
   CalendarUnit unit;
   /// What day does the week start with (Monday=true, Sunday=false)
   bool week_starts_monday;
+  /// If True times exactly on unit multiple boundary will be rounded up one 
unit.
+  /// This applies for ceiling only.
+  bool ceil_on_boundary;
+  /// By default time is rounded to a multiple of units since 
1970-01-01T00:00:00.
+  /// By setting multiple_since_greater_unit to true, time will be rounded to 
number

Review Comment:
   ```suggestion
     /// By setting multiple_since_greater_unit to true, time will be rounded 
to a number
   ```



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