kumarUjjawal commented on PR #18937:
URL: https://github.com/apache/datafusion/pull/18937#issuecomment-3580268987

   > I don't think the changes are doing what you expect here; I believe 
decimals are still being converted to float for these calculations. What we are 
looking to do is to perform the calculations natively on decimals without 
casting them to floats. The function already supports decimal inputs by casting 
them to floats.
   
   Thanks for pointing that. I just checked out the PR #1408 for the guidance 
and updated the code. I also used LLM for this.(disclosure)


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to