lidavidm commented on PR #12590: URL: https://github.com/apache/arrow/pull/12590#issuecomment-1089568115
> Does the dispatcher automatically expand scalar columns if there is no scalar kernel but there is an array kernel? Do you think we should expose scalar/array shape in this PR? No, we don't automatically expand scalars AFAIK. We could wrap Python kernels in this, or something similar: https://github.com/apache/arrow/blob/9fa34df27eb1445ac11b0ab0298d421b04be80f7/cpp/src/arrow/compute/kernels/util_internal.h#L70-L77 This will do the expansion for you. I think we will eventually need to expose the distinction, but if we only support array shape for now, that's probably easier (as long as the interface extends naturally). -- 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]
