Skip to content

Conversation

@Blargian
Copy link
Member

@Blargian Blargian commented Dec 12, 2025

Summary

We have pages for table engines which directly import the content of the table engines pages in the reference docs section.

Problems with this approach:

  • This section is not kept up to date, so we are missing integrations.
  • If we move or modify (rename, split into multiple pages etc) the source page we now need to update this page as well (as in Docs: reorganise iceberg reference docs ClickHouse#91778). This adds an unnecessary dependency.
  • Makes search results worse because you now get two results for the same content

Importing entire pages into other pages is generally not a best practice. This should be limited to snippets of content and we should prefer to cross-link in this case to keep things clean.

This PR:

  • deduplicates this content by replacing these pages with a link to the integration table engines section.
  • adds redirects from those pages to the appropriate source page in the reference section

Checklist

@Blargian Blargian requested review from a team as code owners December 12, 2025 16:22
@vercel
Copy link

vercel bot commented Dec 12, 2025

@Blargian is attempting to deploy a commit to the ClickHouse Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Dec 15, 2025

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

Project Deployment Review Updated (UTC)
clickhouse-docs Ready Ready Preview Dec 15, 2025 0:02am
2 Skipped Deployments
Project Deployment Review Updated (UTC)
clickhouse-docs-ru Ignored Ignored Preview Dec 15, 2025 0:02am
clickhouse-docs-zh Ignored Ignored Preview Dec 15, 2025 0:02am

@Blargian
Copy link
Member Author

I'm going to merge this as it's blocking me on another PR. If there are strong feelings about this change we can always revert.

@Blargian Blargian merged commit 2f917cf into ClickHouse:main Dec 18, 2025
14 checks passed
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.

1 participant