Skip to content

Fix differential CI with ClickHouse 26.9 - #15

Merged
ShawnChen-Sirius merged 1 commit into
chdb-io:mainfrom
ShawnChen-Sirius:fix/clickhouse-26.9-differential-ci
Sep 30, 2026
Merged

ShawnChen-Sirius merged 1 commit into
chdb-io:mainfrom
ShawnChen-Sirius:fix/clickhouse-26.9-differential-ci

Conversation

@ShawnChen-Sirius

@ShawnChen-Sirius ShawnChen-Sirius commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • avoid ClickHouse 26.9 parsing INSERT ... VALUES seed data as TSV
  • append SQL-native FORMAT to the final query for output formatting
  • add a regression test for the differential runner

Validation

  • ruff and mypy pass
  • 315 unit/contract tests pass
  • 25 differential tests pass against ClickHouse 26.5
  • 24/24 differential scenarios pass against ClickHouse 26.9.2

Fixes the scheduled differential CI failure.

Note

Fix differential CI with ClickHouse 26.9 by moving output format to SQL

The ClickHouse 26.9 release makes the CLI --format option also control INSERT input parsing, which breaks multi-statement SQL blocks with inline VALUES. ClickHouseLocal.run in conftest.py now trims trailing semicolons and appends a SQL-level FORMAT clause instead of passing --format to the subprocess. Adds a subprocess-mocked test in test_differential_runner.py that asserts the appended FORMAT TabSeparated and the removed CLI flag.

Macroscope summarized 41680fe.

@ShawnChen-Sirius
ShawnChen-Sirius merged commit 133ebbf into chdb-io:main Sep 30, 2026
15 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.

1 participant