fix(ci): use clickhouse-client for multi-statement SQL init #1819
Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
v2-docs | f1783b8 | Jan 07 2026, 07:28 AM |
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
koxudaxi
force-pushed
the
01-07-fix_ci_use_clickhouse-client_for_multi-statement_sql_init
branch
from
14eb26e to
d4fea98
Compare
koxudaxi
force-pushed
the
01-06-feat_docker_add_clickhouse_service_for_local_development
branch
from
680a1a7 to
7c1dba9
Compare
koxudaxi
force-pushed
the
01-07-fix_ci_use_clickhouse-client_for_multi-statement_sql_init
branch
from
d4fea98 to
78facee
Compare
koxudaxi
force-pushed
the
01-06-feat_docker_add_clickhouse_service_for_local_development
branch
from
7c1dba9 to
396eb63
Compare
koxudaxi
force-pushed
the
01-07-fix_ci_use_clickhouse-client_for_multi-statement_sql_init
branch
2 times, most recently
from
aeeee53 to
901c620
Compare
koxudaxi
changed the base branch from
01-06-feat_docker_add_clickhouse_service_for_local_development
to
graphite-base/1819
koxudaxi
force-pushed
the
01-07-fix_ci_use_clickhouse-client_for_multi-statement_sql_init
branch
from
901c620 to
2a43639
Compare
koxudaxi
force-pushed
the
graphite-base/1819
branch
from
396eb63 to
6dfcc0a
Compare
koxudaxi
changed the base branch from
graphite-base/1819
to
01-06-feat_docker_add_clickhouse_service_for_local_development
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
| echo "Running $script..." | ||
| curl -s "http://${CLICKHOUSE_USER}:${CLICKHOUSE_PASSWORD}@localhost:8123/" --data-binary @"$script" | ||
| done | ||
| run: bun run clickhouse:migrate |
There was a problem hiding this comment.
This line is causing the CI to fail because it's using 'bun run clickhouse:migrate' which is likely not properly set up. Either the script is not defined in package.json, or the migrator.ts file and migrations directory with SQL files don't exist in the expected location. Ensure that the clickhouse:migrate script is defined in package.json, the migrator.ts file is properly implemented, and the SQL migration files have been moved from docker/clickhouse/init/ to clickhouse/migrations/.
Spotted by Graphite Agent (based on CI logs)
Is this helpful? React 👍 or 👎 to let us know.
| echo "Running $script..." | ||
| curl -s "http://${CLICKHOUSE_USER}:${CLICKHOUSE_PASSWORD}@localhost:8123/" --data-binary @"$script" | ||
| done | ||
| run: bun run clickhouse:migrate |
There was a problem hiding this comment.
This line is causing CI to fail because the new 'bun run clickhouse:migrate' command is trying to execute migration files that contain PostgreSQL-style syntax incompatible with ClickHouse. The migration files need to be updated to use proper ClickHouse syntax (replacing types like VARCHAR with String, etc.) before this command can work correctly.
Spotted by Graphite Agent (based on CI logs)
Is this helpful? React 👍 or 👎 to let us know.
koxudaxi
force-pushed
the
01-07-fix_ci_use_clickhouse-client_for_multi-statement_sql_init
branch
from
2a43639 to
b353486
Compare
koxudaxi
force-pushed
the
01-06-feat_docker_add_clickhouse_service_for_local_development
branch
from
6dfcc0a to
e2314ff
Compare
koxudaxi
force-pushed
the
01-07-fix_ci_use_clickhouse-client_for_multi-statement_sql_init
branch
2 times, most recently
from
5b459f4 to
5a1fe76
Compare
koxudaxi
force-pushed
the
01-06-feat_docker_add_clickhouse_service_for_local_development
branch
from
e2314ff to
9445846
Compare
- Replace CI ClickHouse init with `bun run clickhouse:migrate` - Remove docker/clickhouse/init/ directory (migrator handles schema) - Remove init volume mount from compose.yml - Update STRUCTURE.md to reflect migrator-based setup 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
koxudaxi
force-pushed
the
01-07-fix_ci_use_clickhouse-client_for_multi-statement_sql_init
branch
from
5a1fe76 to
f1783b8
Compare
koxudaxi
force-pushed
the
01-06-feat_docker_add_clickhouse_service_for_local_development
branch
from
9445846 to
f5af9b1
Compare
koxudaxi
changed the base branch from
01-06-feat_docker_add_clickhouse_service_for_local_development
to
graphite-base/1819
The consolidated 00_init.sql contains multiple statements (CREATE DATABASE,
CREATE TABLE, ALTER TABLE). ClickHouse's HTTP interface doesn't support
multi-statement queries, so curl --data-binary fails silently after the
first statement.
Solution: Use docker exec with clickhouse-client --multiquery to properly
execute all statements in the init scripts.