Skip to content

Replace Tinybird SQL client with ClickHouse HTTP client - #29

Merged
Makisuo merged 2 commits into
mainfrom
codex/estimate-clickhouse-portability
Apr 29, 2026
Merged

Makisuo merged 2 commits into
mainfrom
codex/estimate-clickhouse-portability

Conversation

@Makisuo

@Makisuo Makisuo commented Apr 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Swap raw SQL execution from @tinybirdco/sdk to @clickhouse/client-web while keeping the TinybirdService facade and call sites intact
  • Add ClickHouse env support with managed defaults and preserve Tinybird-backed ingest/BYO provisioning paths
  • Normalize legacy SQL before execution so existing queries do not double-append FORMAT
  • Extend routing coverage for managed config, org overrides, JSONEachRow execution, and format stripping

Testing

  • Added and updated unit tests for ClickHouse client selection, SQL routing, OrgId guards, and query normalization
  • Ran the focused API service test suite for TinybirdRouting.test.ts

@Makisuo
Makisuo force-pushed the codex/estimate-clickhouse-portability branch from a19b8ad to c473821 Compare April 29, 2026 22:43
@Makisuo
Makisuo marked this pull request as ready for review April 29, 2026 22:56
@Makisuo
Makisuo merged commit 26324a5 into main Apr 29, 2026
1 of 3 checks passed
@pullfrog

pullfrog Bot commented Apr 29, 2026

Copy link
Copy Markdown
Contributor

New pull request. Leaping into action...

Pullfrog  | View workflow run | via Pullfrog | Using Claude Opus | 𝕏

@Makisuo
Makisuo deleted the codex/estimate-clickhouse-portability branch April 29, 2026 22:57
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