Skip to content

Fix timeout retries - #508

Open
Str1kez wants to merge 4 commits into
python-arq:mainfrom
Str1kez:fix-retry-on-timeout
Open

Fix timeout retries#508
Str1kez wants to merge 4 commits into
python-arq:mainfrom
Str1kez:fix-retry-on-timeout

Conversation

@Str1kez

@Str1kez Str1kez commented Sep 28, 2025

Copy link
Copy Markdown

I noticed that with retries enabled, there are no timeout retries.
I read the documentation, and it says that wait_for throws a TimeoutError, not a CancelledError.

@codecov-commenter

codecov-commenter commented Sep 28, 2025

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main     #508      +/-   ##
==========================================
- Coverage   96.27%   96.10%   -0.18%     
==========================================
  Files          11       11              
  Lines        1074     1078       +4     
  分支      209      190      -19     
==========================================
+ Hits         1034     1036       +2     
- Misses         19       21       +2     
  Partials       21       21              
Files with missing lines Coverage Δ
arq/worker.py 97.37% <100.00%> (+0.20%) ⬆️

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a911f3...bdf2f1f. Read the comment docs.

🚀 新建 features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

注册 for free to join this conversation on GitHub. Already have an account? 登录 to comment

标签

None yet

项目

None yet

Development

Successfully merging this pull request may close these issues.

2 participants