🐛 Source Shopify: Add retry job for status ConnectionErrors [issue 7084]

[strosek]

What

Shopify connector is affected by Connection Reset/Aborted errors which wrongly show in traces as system errors and don't have a proper error message. See issue

How

We solved this by making such type of errors retry-able, so these errors don't affect sync jobs immediately.

Review guide

Let's make sure the solution is complete and tests are meaningful. job.py - Added ConnectionError to the list of handled exceptions for retry. base_streams.py - Added condition for 104 status code with retry True. Relevant test cases in test_job.py and unit_test.py, respectively.

User Impact

The user should not see errors related to ConnectionReset/ConnectionAborted as these errors are retried, unless connection is seriously bad.

Can this PR be safely reverted and rolled back?

YES 💚

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
airbyte-docs	⬜️ Ignored (Inspect)	Visit Preview		Apr 25, 2024 6:25pm

[bazarnov]

Closing as fixed.