CuteChuanChuan commented on PR #21988: URL: https://github.com/apache/datafusion/pull/21988#issuecomment-4381079966
Hi @Jefffrey, thanks for raising this — it's a good observation. My intuition is that if a user chose Decimal for their column, they likely expect the median to stay in Decimal as well, since Decimal preserves exact precision whereas Float64 is an approximation. Under that assumption, I'd lean toward keeping the current behavior in this PR. That said, I'm not fully sure this assumption holds for every use case — some users may prefer uniform Float64 output, which is the `percentile_cont` direction. It's a genuine design question, and I'm happy to follow whichever direction the community prefers. Curious to hear your and others' thoughts. -- 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]
