Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't spam logs about unknown types being sent as text #1291

Merged
merged 13 commits into from
Feb 14, 2024

Conversation

serprex
Copy link
Contributor

@serprex serprex commented Feb 14, 2024

Logs of this nature should only be generated once per connector

PostgresCDCSource & QRepQueryExecutor now extend PostgresConnector,
stops QRepQueryExecutor querying for customTypesMapping for snapshots

Logs of this nature should only be generated once per connector

PostgresCDCSource & QRepQueryExecutor now extend PostgresConnector,
stops QRepQueryExecutor querying for customTypesMapping for snapshots
@serprex serprex force-pushed the less-noisy-type-warning branch from 6b0c371 to ed44388 Compare February 14, 2024 15:53
…r we have proper parsing of pg types now in tests
@serprex serprex force-pushed the less-noisy-type-warning branch from 4f067d7 to 12c15be Compare February 14, 2024 17:00
@serprex serprex force-pushed the less-noisy-type-warning branch from 9a7e345 to a4aa925 Compare February 14, 2024 17:33
@serprex serprex force-pushed the less-noisy-type-warning branch 2 times, most recently from 64b89cd to 8257239 Compare February 14, 2024 18:32
@serprex serprex force-pushed the less-noisy-type-warning branch from 8257239 to bcb3636 Compare February 14, 2024 18:34
@serprex serprex force-pushed the less-noisy-type-warning branch from 3817863 to 0b1c49f Compare February 14, 2024 18:49
@serprex serprex force-pushed the less-noisy-type-warning branch from b5b522a to d1b9b63 Compare February 14, 2024 19:27
@serprex serprex merged commit 758727a into main Feb 14, 2024
7 checks passed
@serprex serprex deleted the less-noisy-type-warning branch February 14, 2024 19:47
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.

3 participants