felipecrv commented on PR #13752: URL: https://github.com/apache/arrow/pull/13752#issuecomment-1379238992
The original function seemed to already imply it only worked at the scalar type level. I've seen this function used to switch on the DataType of Datum when selecting the implementation of a kernel, so I don't think adding a second version is necessary. Maybe a rename, but I guess "type" when by itself doesn't include array types, but the type of the scalar or of the individual values that make up the array. -- 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]
