Skip to content

[WIP] 831 no default-case observability for unhandled binlog event types#4456

Draft
ilidemi wants to merge 2 commits into
mainfrom
mysql/task-831
Draft

[WIP] 831 no default-case observability for unhandled binlog event types#4456
ilidemi wants to merge 2 commits into
mainfrom
mysql/task-831

Conversation

@ilidemi

@ilidemi ilidemi commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

80%

@codecov

codecov Bot commented Jun 19, 2026

Copy link
Copy Markdown

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
2335 1 2334 197
View the top 1 failed test(s) by shortest run time
github.com/PeerDB-io/peerdb/flow/connectors/postgres::TestRunPipeline_FilterStripsLines
Stack Traces | 0.01s run time
=== RUN   TestRunPipeline_FilterStripsLines
    pgdump_schema_test.go:262: unexpected error: src failed: signal: killed
        stderr:
--- FAIL: TestRunPipeline_FilterStripsLines (0.01s)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@github-actions

Copy link
Copy Markdown
Contributor

🔄 Flaky Test Detected

Analysis: The DB-independent unit test TestRunPipeline_FilterStripsLines failed with "src failed: signal: killed" — an external SIGKILL of a trivial printf process indicating runner OOM/resource exhaustion, not a code bug, especially since the same package passed in the sibling maria matrix and the failure is unrelated to the PR's MySQL binlog change.
Confidence: 0.88

✅ Automatically retrying the workflow

View workflow run

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can remove tests

@ilidemi

ilidemi commented Jun 22, 2026

Copy link
Copy Markdown
Contributor Author

double check: metric declaration, points where it's emitted, warningReported logic

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