raghavyadav01 commented on PR #26640: URL: https://github.com/apache/datafusion/pull/26640#issuecomment-6389035061
Rebased past #26551 — no conflict; that change is in the `Filter` arm of `PullUpCorrelatedExpr` and this one is in `Limit`. Optimizer tests pass, and the one `subquery_sort.slt` failure reproduces identically on untouched main. @jayzhan211 would you have time to look? You merged #26551 and #25781 in this area. The change is narrow: `EXISTS` currently drops an inner `LIMIT` unconditionally, which is wrong when a node above the `LIMIT` can empty the subquery on its own — `subquery.slt` already asserts the `offset 1` case should fail rather than return wrong rows, and this extends that reasoning to `fetch`. -- 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]
