discord9 commented on issue #17166: URL: https://github.com/apache/datafusion/issues/17166#issuecomment-3183731923
> [Git bisect](https://git-scm.com/docs/git-bisect) might help to find the PR that fixes it. > > I think it's also a good idea to include the reproducer to the test, if we haven't done that in the fixing PR yet. it appear that this pr fc2f9dd8cc279a7a3f53509bae3fa275a0585509 https://github.com/apache/datafusion/pull/14860 "fixed" it, I don't think that's intentional, becuase this pr just default to not `add_implicit_group_by_exprs`? and even if I set `add_implicit_group_by_exprs=true` then the bug also wouldn't appear, so I'm still not sure how does this pr fix 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
