raghavyadav01 commented on issue #24960: URL: https://github.com/apache/datafusion/issues/24960#issuecomment-6612563510
@alamb would it be ok if I take the remaining shape here? It still returns wrong rows on main (198ab43f8): - `EXISTS (... HAVING count(*) + t1.t1_int > 1)` returns `2`, DuckDB returns `2, 3` - the `NOT EXISTS` form returns `1, 3`, DuckDB returns `1` The plan is to make this shape error, as you suggested, not to make it work. #26695 (HAVING on outer columns only) is close and @mohammadnaqvi04 has it. I'd keep to the count + outer column case so we don't overlap, unless you'd rather one PR covers both. -- 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]
