ozankabak commented on PR #14699: URL: https://github.com/apache/datafusion/pull/14699#issuecomment-2676934776
Thanks for all the comments and questions. I've incorporated the naming suggestion by @alamb (and updated many comments and variable names accordingly). I also switched to `GenericDistribution` instead of `UnknownDistribution` per our discussion with @comphead, just to make sure we avoid any confusion. I think this is good to go -- we can proceed with the follow-up tasks, which are: (1) higher-level `ColumnStatistics`/`TableStatistics` revamps, (2) adding sampling support, (3) incrementally supporting more distributions and their interactions, (4) integrations with the optimizer code. I will wait for a day or so for more feedback in case there is any that we missed. -- 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
