edponce commented on pull request #11113: URL: https://github.com/apache/arrow/pull/11113#issuecomment-918234353
cc @lidavidm The floating-point kernels need to return a floating-point type because of `sign(NaN) = NaN` and NaN is not support in integer types. For the integral kernels, Int8 is returned because output can be any of (-1,0,1). I was thinking last week of tests for code coverage. I actually know of several snippets of dead code. -- 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]
