abreis commented on pull request #9454: URL: https://github.com/apache/arrow/pull/9454#issuecomment-779404775
Given that `compute::kernels::comparison` already makes use liberal use of `<op>_scalar` methods, I'd like to propose the following as a way forward for this PR: 1. Merge `divide_scalar` as-is, and other eventual PRs for arithmetic scalar ops (`add_scalar`, etcetera). 2. Propose the use of `Datum` in a separate PR as a way to generalize kernel inputs for arrays and scalars. 3. If accepted, make every `<op>_scalar` method in the codebase simply delegate to its (now `Datum`-using) base method. 4. Mark the `_scalar` methods with `#[deprecated]`. Remove them altogether in future major release. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
