asolimando commented on PR #20926: URL: https://github.com/apache/datafusion/pull/20926#issuecomment-4073101595
> Thanks @asolimando for detailed review. I thought this one was more polished after looking at Trino and Spark in detail but I've missed some important points. I'll carefully apply your reviews and adjust implementation early tomorrow No worries, I think the PR already brings a considerable improvement over the existing, and that the final outcome will be a more precise estimation than Trino and Spark. I am bit busy today and tomorrow so don't rush unless it's good for your schedule. I will make sure to review changes by the end of the week. -- 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]
