-
Notifications
You must be signed in to change notification settings - Fork 93
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
zmq: (don't) re-implement task message retries? #3331
Comments
Result of offline discussion with @matthewrmshin: maybe we shouldn't do this? The retry will almost always fail anyway. |
Yeah I recall a discussion (with @matthewrmshin) of this somewhere else recently (but I don't recall where). I tend to agree - message retries are almost always a waste of time, so we shouldn't bother. Polling (job query) is the only effective way to catch/recover-from status messaging failures. |
@oliver-sanders - can you check that I've got everything (in PR #3402) ... your |
Re: the conversation on removing task message retries was this comment: |
I think I added a TODO in the code with "see #3331", seems to have been removed by another change. |
See also #2978
Simple feature removed in the initial Python3 port. For more information see:
The text was updated successfully, but these errors were encountered: