mohitgurav20 commented on PR #25719: URL: https://github.com/apache/datafusion/pull/25719#issuecomment-6014101875
@asolimando I've gone through all the threads carefully and addressed everything: Added an end-to-end test in subquery.slt that runs EXPLAIN with show_statistics = true on a WHERE name = (SELECT 'a') query — the physical plan output now explicitly shows Rows=Inexact(1) on the FilterExec, demonstrating the NDV-based estimate is working. Coverage is now at 100% (Codecov shows all modified lines covered). Switched to normal commits instead of force-pushing, so the diff stays reviewable. Sorry for the noise from the earlier force-pushes. Happy to hear any further feedback! -- 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]
