Skip to content

Add support for SYSTEM RELOAD DICTIONARY ... ON CLUSTER - #308

Merged
git-hulk merged 1 commit into
AfterShip:masterfrom
vincentbernat:fix/system-reload-on-cluster
Aug 6, 2026
Merged

Add support for SYSTEM RELOAD DICTIONARY ... ON CLUSTER#308
git-hulk merged 1 commit into
AfterShip:masterfrom
vincentbernat:fix/system-reload-on-cluster

Conversation

@vincentbernat

Copy link
Copy Markdown
Contributor

SYSTEM RELOAD DICTIONARY accept an ON CLUSTER clause, but the parser rejected it:

SYSTEM RELOAD DICTIONARY ON CLUSTER akvorado akvorado.asns
SYSTEM RELOAD DICTIONARIES ON CLUSTER akvorado

ClickHouse accepts the clause both before and after the dictionary name and formats both back with the cluster first, so the parser now does the same.

SYSTEM RELOAD DICTIONARY accept an ON CLUSTER clause, but the parser
rejected it:

  SYSTEM RELOAD DICTIONARY ON CLUSTER akvorado akvorado.asns
  SYSTEM RELOAD DICTIONARIES ON CLUSTER akvorado

ClickHouse accepts the clause both before and after the dictionary name
and formats both back with the cluster first, so the parser now does the
same.
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Repo admins can enable using credits for code reviews in their settings.

@vincentbernat
vincentbernat force-pushed the fix/system-reload-on-cluster branch from 206f0de to b22db3f Compare August 5, 2026 13:37

@git-hulk git-hulk left a comment

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.

LGTM

@git-hulk
git-hulk merged commit 5dd25f8 into AfterShip:master Aug 6, 2026
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.

2 participants