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

Timeout in user defined handler should cause retry #907

Closed
Pijukatel opened this issue Jan 15, 2025 · 0 comments · Fixed by #909
Closed

Timeout in user defined handler should cause retry #907

Pijukatel opened this issue Jan 15, 2025 · 0 comments · Fixed by #909
Labels
bug Something isn't working. t-tooling Issues with this label are in the ownership of the tooling team.

Comments

@Pijukatel
Copy link
Contributor

Pijukatel commented Jan 15, 2025

Timeouts in user defined handlers should be treated the same way as other exceptions raised from within the user handler.
This should cause retry.

Currently the request that times out will end up in strange state in RequestQueue and it will block crawler from closing for 300 secs until following triggers:
"The request queue seems to be stuck for 300.0s, resetting internal state."

(Regarding blocking sync code timeouts from user defined handlers: #908)

@Pijukatel Pijukatel added bug Something isn't working. t-tooling Issues with this label are in the ownership of the tooling team. labels Jan 15, 2025
@Pijukatel Pijukatel changed the title Timeout in user defined handler should cause retry Timeout in user defined handler should cause retry (async blocking code) Jan 15, 2025
@Pijukatel Pijukatel changed the title Timeout in user defined handler should cause retry (async blocking code) Timeout in user defined handler should cause retry Jan 15, 2025
@vdusek vdusek added this to the 106th sprint - Tooling team milestone Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working. t-tooling Issues with this label are in the ownership of the tooling team.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants