Skip to content

perf(bridge): Q1.3 ClickHouse-only batch apply#178

Merged
brownjuly2003-code merged 5 commits into
mainfrom
q13-ch-batch-apply
Jul 9, 2026
Merged

perf(bridge): Q1.3 ClickHouse-only batch apply#178
brownjuly2003-code merged 5 commits into
mainfrom
q13-ch-batch-apply

Conversation

@brownjuly2003-code

Copy link
Copy Markdown
Owner

Summary

Q1.3 write-path: production serving bridge is ClickHouse only.

  • apply_serving_batch — multi-row orders_v2 + pipeline_events
  • users_enriched recompute once per unique user (not per order)
  • Journal written last (crash mid-batch stays replayable)
  • DuckDB is demo/unit only; never used when CH sink is set

Why DuckDB appeared before

Historical dual-write: bridge reused _process_event which always wrote a local lake then mirrored to CH. That scratch lake was never the API store. Q1.2 skipped it; Q1.3 batches pure CH writes.

Verify

45 passed — bridge + pipeline unit (Windows)

Test plan

  • Unit: batch one multi-row insert, unique user aggregates
  • CI green
  • Mac S10 re-measure after merge

JuliaEdom added 2 commits July 9, 2026 21:04
Production bridge path applies events via apply_serving_batch: multi-row
orders_v2 and pipeline_events inserts, users_enriched once per unique user,
journal last for replay safety. DuckDB is demo/unit only — never opened
when the ClickHouse sink is set.
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

DORA Metrics

  • Window: last 30 days
  • Branch: main
  • Deployment frequency: 98 total / 22.87 per week
  • Lead time for changes: avg 0.4h / median 0.0h
  • Change failure rate: 61.22% (60/98)
  • MTTR: 2.56h across 1 incident(s)

@brownjuly2003-code brownjuly2003-code merged commit 4aa03be into main Jul 9, 2026
22 checks passed
@brownjuly2003-code brownjuly2003-code deleted the q13-ch-batch-apply branch July 9, 2026 18:26
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