Skip to content

Fix/clickhouse normalize cursor skip - #4675

Open
matheusgr wants to merge 3 commits into
PeerDB-io:mainfrom
talismanai:fix/clickhouse-normalize-cursor-skip
Open

Fix/clickhouse normalize cursor skip#4675
matheusgr wants to merge 3 commits into
PeerDB-io:mainfrom
talismanai:fix/clickhouse-normalize-cursor-skip

Conversation

@matheusgr

Copy link
Copy Markdown

No description provided.

ilidemi and others added 2 commits June 29, 2026 14:30
… tables

When normalize finds no destination tables (connections=0) but the batch
range still has rows in the raw table or a non-empty avro stage, return an
error instead of calling UpdateNormalizeBatchID. This lets the normalize
loop retry and prevents permanently skipping CDC batches.

Empty CDC batches (zero avro records and zero raw rows) still advance the
cursor as before.

Co-authored-by: Cursor <cursoragent@cursor.com>
@matheusgr
matheusgr requested a review from a team as a code owner August 6, 2026 16:30
@matheusgr
matheusgr requested a deployment to external-contributor August 6, 2026 16:30 — with GitHub Actions Waiting
@matheusgr
matheusgr requested a deployment to external-contributor August 6, 2026 16:30 — with GitHub Actions Waiting
@CLAassistant

CLAassistant commented Aug 6, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@matheusgr
matheusgr requested a deployment to external-contributor August 6, 2026 16:49 — with GitHub Actions Waiting
@matheusgr
matheusgr requested a deployment to external-contributor August 6, 2026 16:49 — with GitHub Actions Waiting
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.

4 participants