Skip to content
This repository was archived by the owner on Nov 7, 2025. It is now read-only.

Conversation

@jakozaur
Copy link
Contributor

ALTER TABLE can be expensive in ClickHouse, it's easier and better to combine them. The MVP is to combine adding a new column with adding a comment to it.

@jakozaur jakozaur requested a review from a team as a code owner June 12, 2025 13:23
@avelanarius avelanarius enabled auto-merge June 12, 2025 14:24
@avelanarius avelanarius added this pull request to the merge queue Jun 12, 2025
Merged via the queue into main with commit 2506adb Jun 12, 2025
7 checks passed
@avelanarius avelanarius deleted the jacek-combine-alter branch June 12, 2025 14:29
nablaone added a commit that referenced this pull request Jun 24, 2025
github-merge-queue bot pushed a commit that referenced this pull request Jun 24, 2025
This reverts commit 2506adb.


Combined alter table statements may not always work. 
```
Error executing ingest statement: code: 48, message: For Replicated databases it's not allowed to execute ALTERs of different types (replicated and non replicated) in single query, query: ALTER TABLE "phone_home_logs" ADD COLUMN IF NOT EXISTS "request_id" Nullable(String), COMMENT COLUMN...
```


<!-- A note on testing your PR -->
<!-- Basic unit test run is executed against each commit in the PR.
If you want to run a full integration test suite, you can trigger it by
commenting
     with '/run-integration-tests' or '/run-it' -->
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants