Skip to content

postgresql cdc v2#4300

Draft
serprex wants to merge 2 commits into
mainfrom
v2-take-2
Draft

postgresql cdc v2#4300
serprex wants to merge 2 commits into
mainfrom
v2-take-2

Conversation

@serprex
Copy link
Copy Markdown
Member

@serprex serprex commented May 13, 2026

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

❌ Test Failure

Analysis: Test_PartitionBy fails deterministically across all 4 test suites with a consistent assertion mismatch (expected partition key "num" but got "(num)"), indicating a real behavioral change — likely a newer ClickHouse version now wraps PARTITION BY expressions in parentheses.
Confidence: 0.9

⚠️ This appears to be a real bug - manual intervention needed

View workflow run

@serprex serprex mentioned this pull request May 13, 2026
Closed
@github-actions
Copy link
Copy Markdown
Contributor

❌ Test Failure

Analysis: Test_CDC_V2_Protocol deterministically times out (200s) across all matrix jobs with other.NumRecords stuck at 1 instead of 2, indicating a real replication bug likely related to the ClickHouse ORDER BY/PARTITION BY change only partially fixed in the previous commit.
Confidence: 0.72

⚠️ This appears to be a real bug - manual intervention needed

View workflow run

@github-actions
Copy link
Copy Markdown
Contributor

🔄 Flaky Test Detected

Analysis: Test_CDC_V2_Protocol times out at ~200s across all three matrix variants (199.33s–200.26s), indicating it's hitting an internal deadline rather than a real assertion failure, causing the overall e2e suite to exhaust the 900s timeout.
Confidence: 0.93

✅ Automatically retrying the workflow

View workflow run

@github-actions
Copy link
Copy Markdown
Contributor

🔄 Flaky Test Detected

Analysis: All failures are timeout errors ("UNEXPECTED TIMEOUT waiting on initial", "UNEXPECTED TIMEOUT v2 single insert", "context deadline exceeded" on Temporal) across all three CI matrix jobs, indicating overloaded CI runners rather than a code regression.
Confidence: 0.92

✅ Automatically retrying the workflow

View workflow run

@github-actions
Copy link
Copy Markdown
Contributor

❌ Test Failure

Analysis: Test_CDC_V2_Protocol fails deterministically across all 3 matrix configurations (~200s each), consistent with a real regression likely introduced by the recent ClickHouse PARTITION BY/ORDER BY change rather than a flaky environment issue.
Confidence: 0.82

⚠️ This appears to be a real bug - manual intervention needed

View workflow run

@github-actions
Copy link
Copy Markdown
Contributor

🔄 Flaky Test Detected

Analysis: The e2e test suite for the maria/8.0 matrix job hit the hard 900-second timeout and was killed mid-run, with no assertion failures, indicating infrastructure slowness or a stuck test rather than a code regression.
Confidence: 0.88

✅ Automatically retrying the workflow

View workflow run

@github-actions
Copy link
Copy Markdown
Contributor

❌ Test Failure

Analysis: Test_NullEngine deterministically fails in all 4 test suites (MySQL_CH, MySQL_CH_Cluster, PG_CH, PG_CH_Cluster) because the ClickHouse NullEngine table never accumulates records (other.NumRecords stays at 0 across every polling cycle), which is likely a real regression from the recent ClickHouse version behavior change (the prior commit already patched PARTITION BY/ORDER BY issues from the same upgrade).
Confidence: 0.78

⚠️ This appears to be a real bug - manual intervention needed

View workflow run

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