findepi commented on PR #7070:
URL: https://github.com/apache/arrow-rs/pull/7070#issuecomment-2637276700

   thanks for fixing this!
   
   additional test case would be nice. casting from value 0 of decimal(3,0) 
type to decimal(2,0) type used to produce 1 before this fix. it sounds like a 
test case that should be in the code base (will add)
   
   @himadripal would you want to backport this fix to the `53.0.0_maintenance` 
branch? latest 53.4.0 is affected by this bug
   
   (assuming we would want to release 53.5.0 release. for a correctness bug 
like this i think we should, but not my call)


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