jchen5 commented on PR #40865: URL: https://github.com/apache/spark/pull/40865#issuecomment-1518139762
I checked the case of `any_value(false)` in a debugger and it works because resultWithZeroTups is NULL there, so that explains why it works - because there's an aggregation value around false, and not just constant false. This fix seems to work with the cases I've thought of (results match SQL standard semantics and postgres). I think this PR is mergeable. I'll plan to add some more test coverage for related cases in a future PR. -- 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]
