edponce commented on pull request #11882: URL: https://github.com/apache/arrow/pull/11882#issuecomment-1003110634
From the previous discussion on compute function's API, recall that the main entry point for invoking a function is `CallFunction(name, args, options, ctx)`. This allows passing function-specific `options`. If functions were to be grouped and the `options` parameter used to select a particular function, then expressing function-specific `options` would be difficult. For this particular kernels, `between`, I recommend to have a single `between` function that uses the `options` from `CallFunction` to select the desired kernel implementation. This approach will provide the structure for the `not_between` function as well. @bkmgit I created ARROW-15223 as a follow-up JIRA for adding the `not_between` compute function. Thanks! -- 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]
