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