bkmgit commented on a change in pull request #11882:
URL: https://github.com/apache/arrow/pull/11882#discussion_r786979074



##########
File path: cpp/src/arrow/compute/api_scalar.h
##########
@@ -342,6 +342,17 @@ struct ARROW_EXPORT CompareOptions {
   enum CompareOperator op;
 };
 
+class ARROW_EXPORT BetweenOptions : public FunctionOptions {
+ public:
+  enum Inclusive { BOTH, LEFT, RIGHT, NEITHER };
+  explicit BetweenOptions(Inclusive inclusive = BOTH);
+  static BetweenOptions Defaults() { return BetweenOptions(); }
+  constexpr static char const kTypeName[] = "BetweenOptions";
+
+  /// Inclusive option to apply

Review comment:
       Updated to
   ``` 
     /// Indicate boundary points to include, both (left <= value <= right), 
     /// left (left <= value < right) , right (left < value <= right)
     /// or neither (left < value < right)
   ```




-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to