Skip to content

Docs: correct join_algorithm default (not auto)#6450

Merged
Blargian merged 1 commit into
mainfrom
fix-docs-join-algorithm-default
Jun 25, 2026
Merged

Docs: correct join_algorithm default (not auto)#6450
Blargian merged 1 commit into
mainfrom
fix-docs-join-algorithm-default

Conversation

@SamWolfberg

Copy link
Copy Markdown
Contributor

Summary

The "Minimize and optimize JOINs" best-practices page states that join_algorithm = 'auto' is the default. The actual default in source is direct,parallel_hash,hash — see src/Core/Settings.cpp (DECLARE(JoinAlgorithm, join_algorithm, "direct,parallel_hash,hash", ...)).

This corrects the statement and notes the real default and what it means (direct join when the right-hand side is a dictionary/key-value engine, otherwise parallel hash, then hash). One-sentence prose change; no URL/anchor change.

Checklist

  • Correction verified against ClickHouse/ClickHouse source

The default in source (ClickHouse/ClickHouse src/Core/Settings.cpp) is
`direct,parallel_hash,hash`, not `auto`.
@SamWolfberg SamWolfberg requested a review from a team as a code owner June 25, 2026 20:57
@vercel

vercel Bot commented Jun 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
clickhouse-docs Ready Ready Preview, Comment Jun 25, 2026 9:09pm
clickhouse-docs-jp Building Building Preview, Comment Jun 25, 2026 9:09pm
3 Skipped Deployments
Project Deployment Actions Updated (UTC)
clickhouse-docs-ko Ignored Ignored Preview Jun 25, 2026 9:09pm
clickhouse-docs-ru Ignored Ignored Preview Jun 25, 2026 9:09pm
clickhouse-docs-zh Ignored Ignored Preview Jun 25, 2026 9:09pm

Request Review

@mintlify

mintlify Bot commented Jun 25, 2026

Copy link
Copy Markdown

Docs PR opened: ClickHouse/mintlify-docs-dev#281

Corrected the JOINs best practices page to show the real join_algorithm default is direct,parallel_hash,hash instead of auto.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants