-
Notifications
You must be signed in to change notification settings - Fork 4k
Description
Supporting scalar outputs from array kernels (where all the inputs are scalars) introduces needless complexity into the kernel implementations, causing duplication of effort and excess code generation for paltry benefit. In the scenario where all inputs are scalars, it would be better to promote them all to arrays of length 1 (either by creating the arrays or constructing an appropriate ArraySpan per ARROW-16756) and invoking the array code path. This would enable us to delete thousands of lines of code and ease the ongoing development and maintenance of the array kernels codebase
Reporter: Wes McKinney / @wesm
Assignee: Wes McKinney / @wesm
Related issues:
- [C++] Improve array expression and kernel evaluation performance on small inputs (relates to)
- [C++] ValueDescr::SCALAR nearly unused and does not work for projection (supercedes)
PRs and other links:
Note: This issue was originally created as ARROW-16757. Please see the migration documentation for further details.