edponce edited a comment 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 kernel, `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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
