-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
CDK: make postgres temp_file_limit error a transient error #38667
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
e.message!!.lowercase().contains("An I/O error occurred while sending to the backend") | ||
(e.message!!.lowercase().contains("An I/O error occurred while sending to the backend") || | ||
e.message!!.lowercase().contains("temporary file size exceeds temp_file_limit")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be great to start building up a constant or array of these matchers somewhere
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be great to start building up a constant or array of these matchers somewhere
Agreed.
closing this to send PR from my branch of upstream repo instead of using my own fork. (to avoid the community-ci failure issue) |
Fixes #27090
A postgres throws a
temp_file_limit
error is translated to a system error for airbyte. I believe this is true for both postgres source and destination connectors.This change makes this error a transient error so that it won't trigger sentry.