alamb opened a new pull request, #16605: URL: https://github.com/apache/datafusion/pull/16605
## Which issue does this PR close? <!-- We generally require a GitHub issue to be filed for all bug fixes and enhancements and this helps us generate change logs for our releases. You can link an issue to this PR using the GitHub syntax. For example `Closes #123` indicates that this PR will close issue #123. --> - Closes https://github.com/apache/datafusion/issues/16591 - Closes https://github.com/apache/datafusion/pull/16599 ## Rationale for this change I forgot to set an important setting when running the clickbench queries locally with `datafusion-cli` resulting in strange performance results. We can't just change the default value of this setting as it isn't correct in general (see https://github.com/apache/datafusion/pull/16599#issuecomment-3015291713) it is pretty specific to the ClickBench datafiles @zhuqi-lucas proposed changing the default in `datafusion-cli` but I fear that may also cause confusion -- see https://github.com/apache/datafusion/pull/16599 for more details ## What changes are included in this PR? Add comments to the ClickBench queries so future readers will not be (as) likely to make the same mistake I did ## Are these changes tested? I tested locally to make sure the comments are ignored by the benchmark runner and they are ```shell ./benchmarks/bench.sh run clickbench_partitioned 41 ``` And ``` ./benchmarks/bench.sh run clickbench_partitioned ``` To verify that the queries still run correctly with the comments ## Are there any user-facing changes? <!-- If there are user-facing changes then we may require documentation to be updated before approving the PR. --> <!-- If there are any breaking changes to public APIs, please add the `api change` label. --> -- 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
