khwilson commented on PR #44184:
URL: https://github.com/apache/arrow/pull/44184#issuecomment-2402404888

   Thanks for the review!
   
   By a user-side cast, do you mean that users should essentially do:
   
   ```sql
   select avg(cast(blah as decimal(big-precision)))
   ```
   
   instead of
   
   ```sql
   select avg(blah)
   ```
   
   or do you mean that this code should "inject" a cast on the "user" side?
   
   If you mean putting the cast onto the user, then I would think you'd want to 
add an error if the answer can't fit into the default precision, but that seems 
like it would be more disruptive (and out of step with how other systems handle 
decimal aggregates).
   
   If you mean "injecting" the cast on the user side, would that end up 
creating a copy of 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]

Reply via email to