Skip to content

Datalakes catalogs database for distributed processing#88273

Merged
scanhex12 merged 8 commits intomasterfrom
database_catalog_distributed
Oct 14, 2025
Merged

Datalakes catalogs database for distributed processing#88273
scanhex12 merged 8 commits intomasterfrom
database_catalog_distributed

Conversation

@scanhex12
Copy link
Copy Markdown
Member

Changelog category (leave one):

  • Improvement

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

Datalakes catalogs database for distributed processing

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Oct 8, 2025

Workflow [PR], commit [db4384c]

@clickhouse-gh clickhouse-gh bot added the pr-improvement Pull request with some product improvements label Oct 8, 2025
@divanik divanik self-assigned this Oct 8, 2025
const auto can_use_parallel_replicas = !parallel_replicas_cluster_name.empty()
&& query_settings[Setting::parallel_replicas_for_cluster_engines]
&& context_->canUseTaskBasedParallelReplicas()
&& !context_->isDistributed();
Copy link
Copy Markdown
Member

@divanik divanik Oct 9, 2025

Choose a reason for hiding this comment

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

Why do we have this condition? (&& !context_->isDistributed())

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Copied from here

&& !context->isDistributed();

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please, explain its purpose, I don't understand it now

@scanhex12 scanhex12 force-pushed the database_catalog_distributed branch from b019732 to a2cf5ad Compare October 10, 2025 10:56
const auto can_use_parallel_replicas = !parallel_replicas_cluster_name.empty()
&& query_settings[Setting::parallel_replicas_for_cluster_engines]
&& context_->canUseTaskBasedParallelReplicas()
&& !context_->isDistributed();
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please, explain its purpose, I don't understand it now

@scanhex12 scanhex12 enabled auto-merge October 14, 2025 14:20
@scanhex12 scanhex12 added this pull request to the merge queue Oct 14, 2025
Merged via the queue into master with commit d7a93b7 Oct 14, 2025
117 of 123 checks passed
@scanhex12 scanhex12 deleted the database_catalog_distributed branch October 14, 2025 14:35
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-synced-to-cloud The PR is synced to the cloud repo label Oct 14, 2025
@scanhex12 scanhex12 added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label Feb 10, 2026
@robot-clickhouse robot-clickhouse added the pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR label Feb 10, 2026
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Feb 26, 2026
ianton-ru pushed a commit to Altinity/ClickHouse that referenced this pull request Feb 26, 2026
…distributed

Datalakes catalogs database for distributed processing
zvonand added a commit to Altinity/ClickHouse that referenced this pull request Feb 27, 2026
Antalya 25.8 backport of ClickHouse#88273: Datalakes catalogs database for distributed processing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-improvement Pull request with some product improvements pr-must-backport Pull request should be backported intentionally. Use this label with great care! pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants