bkmgit commented on pull request #11720:
URL: https://github.com/apache/arrow/pull/11720#issuecomment-977416868


   Thanks for letting me know. Checking code. Can then close this. Conversion 
is still the same:
   
https://github.com/apache/arrow/blob/master/cpp/src/arrow/compute/kernels/scalar_arithmetic.cc#L252
   Would expect a cast to int32 of the final result so that multiplication of a 
positive and negative number is correct, but maybe this is done elsewhere, as 
the test passes:
   
https://github.com/apache/arrow/blob/master/cpp/src/arrow/compute/kernels/scalar_arithmetic_test.cc#L818


-- 
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]


Reply via email to