edponce commented on pull request #11882: URL: https://github.com/apache/arrow/pull/11882#issuecomment-1002959199
@bkmgit As this PR stands, the `BetweenOptions` are not actually used internally (ie., passed to `CallFunction`) but rather an artifact that only exists if directly using the public scalar API `Between()` which is not the common case. If this PR will not use `BetweenOptions` internally, I would argue for them to not be included nor the public scalar API. These comments depend on the resolution of the previous observations I made wrt to the possible solutions for the `between` API. Please take all these comments as simple observations for discussion, as I had more time to think about this and did not thought about them earlier. -- 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]
