Skip to content

Conversation

alamb
Copy link
Contributor

@alamb alamb commented Aug 4, 2025

Which issue does this PR close?

Rationale for this change

I was trying to add a tuning guide for small data / short queries but got nerd-sniped into fixing
the crate configuration guidelines while finding a place to the tuning guide

What changes are included in this PR?

  1. Make it clear there are several potential build optimizations, and update the explanations
  2. Polish text

Are these changes tested?

By docs CI

Are there any user-facing changes?

Better docs

Copy link
Contributor

@comphead comphead left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @alamb

Copy link
Member

@xudong963 xudong963 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@xudong963 xudong963 merged commit 40015a8 into apache:main Aug 5, 2025
5 checks passed
@alamb alamb deleted the alamb/crate_config_and_tuning_guide branch August 5, 2025 19:42
hknlof pushed a commit to hknlof/datafusion that referenced this pull request Aug 20, 2025
)

* Docs: Update the crate configuration / build settings page

* Update docs/source/user-guide/crate-configuration.md

Co-authored-by: Oleks V <[email protected]>

---------

Co-authored-by: Oleks V <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants