Skip to content

Fix distributed_ddl replicas_path mismatch causing ON CLUSTER DDL to hang#1922

Merged
sunsingerus merged 1 commit intoAltinity:0.26.0from
Elmo33:fix/distributed-ddl-replicas-path
Feb 13, 2026
Merged

Fix distributed_ddl replicas_path mismatch causing ON CLUSTER DDL to hang#1922
sunsingerus merged 1 commit intoAltinity:0.26.0from
Elmo33:fix/distributed-ddl-replicas-path

Conversation

@Elmo33
Copy link
Copy Markdown
Contributor

@Elmo33 Elmo33 commented Feb 10, 2026

The operator generates a custom <distributed_ddl> per CHI (e.g. /clickhouse/{chi-name}/task_queue/ddl) but does not set <replicas_path>. ClickHouse 25.8+ falls back to the default replicas_path from config.xml (/clickhouse/task_queue/replicas), which is in a different subtree. This causes the DDL workers to never pick up tasks, making any CREATE TABLE ... ON CLUSTER hang indefinitely.

Add <replicas_path> alongside so both use the same per-CHI subtree in ZooKeeper/Keeper.

Important items to consider before making a Pull Request

Please check items PR complies to:

  • All commits in the PR are squashed. More info
  • The PR is made into dedicated next-release branch, not into master branch1. More info
  • The PR is signed. More info

--

1 If you feel your PR does not affect any Go-code or any testable functionality (for example, PR contains docs only or supplementary materials), PR can be made into master branch, but it has to be confirmed by project's maintainer.

…hang

The operator generates a custom <distributed_ddl><path> per CHI
(e.g. /clickhouse/{chi-name}/task_queue/ddl) but does not set
<replicas_path>. ClickHouse 25.8+ falls back to the default
replicas_path from config.xml (/clickhouse/task_queue/replicas),
which is in a different subtree. This causes the DDL workers to
never pick up tasks, making any CREATE TABLE ... ON CLUSTER hang
indefinitely.

Add <replicas_path> alongside <path> so both use the same
per-CHI subtree in ZooKeeper/Keeper.
@alex-zaitsev
Copy link
Copy Markdown
Member

It should be only applied for versions >= 25.8.10 to avoid unnecessary restarts. Maybe we can do it via addons.rules configuration instead of the code. Like this:

addons:
    rules:
...
      - version: ">= 25.8.10"
        spec:
          configuration:
            settings:
              distributed_ddl/replicas_path: clickhouse/{chi}/task_queue/replicas

@sunsingerus sunsingerus merged commit 5b2fef5 into Altinity:0.26.0 Feb 13, 2026
1 of 2 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.

3 participants