alamb merged PR #16699:
URL: https://github.com/apache/datafusion/pull/16699
--
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...@datafusi
alamb commented on PR #16699:
URL: https://github.com/apache/datafusion/pull/16699#issuecomment-3045689726
I am happy to make any other suggested changes
--
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
alamb commented on PR #16699:
URL: https://github.com/apache/datafusion/pull/16699#issuecomment-3045688905
Thank you for the review @comphead -- i am going to merge this in ASAP as
we have already released 48.0.1 and it would be nice to have this doc in place
--
This is an automated mess
alamb commented on code in PR #16699:
URL: https://github.com/apache/datafusion/pull/16699#discussion_r2190454962
##
docs/source/library-user-guide/upgrading.md:
##
@@ -137,6 +120,25 @@ SET datafusion.execution.spill_compression = 'zstd';
For more details about this configura
comphead commented on code in PR #16699:
URL: https://github.com/apache/datafusion/pull/16699#discussion_r2190441264
##
docs/source/library-user-guide/upgrading.md:
##
@@ -137,6 +120,25 @@ SET datafusion.execution.spill_compression = 'zstd';
For more details about this config
alamb commented on code in PR #16699:
URL: https://github.com/apache/datafusion/pull/16699#discussion_r2189688564
##
docs/source/index.rst:
##
@@ -126,6 +126,7 @@ To get started, see
:caption: Library User Guide
library-user-guide/index
+ library-user-guide/upgrad
alamb opened a new pull request, #16699:
URL: https://github.com/apache/datafusion/pull/16699
## Which issue does this PR close?
- Part of #16486
## Rationale for this change
We back ported the default setting for statistics
`datafusion.execution.collect_statistics` to
