wiedld commented on PR #14821: URL: https://github.com/apache/datafusion/pull/14821#issuecomment-2686396698
> I think `count()` is constant in this case because there is only one partition perhaps 🤔 > > I think the SQL looks like > > ```sql > SELECT count() over () > ``` > > But maybe not 🤔 The equivalence properties' ordering methods, as well as the `add_sort_above` util method, were checking if the field was a constant -- and not considering if it's a heterogeneous constant. This commit https://github.com/apache/datafusion/pull/14821/commits/5a4ef63d56525f673bcee431c74239de02fd01e3 fixes it. -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
