adriangb commented on PR #16080: URL: https://github.com/apache/datafusion/pull/16080#issuecomment-2992664490
> * See also context [[Potentially] Release DataFusion `48.0.1` #16486](https://github.com/apache/datafusion/issues/16486) > > What @AdamGS did on main was actually switch the defaults so that now statistics are always collected by default, which seems pretty reasonable. It might be that just turning statistics on for everytone on 48.0.1 would be least disruptive rather than changing consistently to off then back to on 🤔 Yep that's what I meant. Personally I think either default is justifiable, but before this PR there were 2 different defaults depending on your usage. But having a single default is inevitably going to have some regressions for some people. I thought that having the default in the session config (`false`) i.e. not collect statistics by default would have less bad of a worst case scenario, but feedback on the other thread seemed to indicate otherwise and I'm happy to accept that - we don't use the default either way. -- 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
