You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is very fitting for use cases where tasks use parameters or data provided by users, for combinations of whom the task may fail in expected ways (that you can't know itwill happen before running it).
In such cases there is no way this may succeed by retrying, there is no value in polluting your sentry/logs with error level info, and you want the caller side to be able to get the resulting error of this task so it can return the appropriate error response.
This also applies to other types of expected errors.
@Bogdanp whould you integrate this feature? I can work on the implementation and provide a PR
The text was updated successfully, but these errors were encountered:
Yes, feel free to work on this. I'd probably just update Retries to recognize a throws option and avoid making changes to the logging since that part of the code won't have full access to those options.
I would like to propose adding one more useful feature that we currently use in celery and will help us finish our migration in a straightforward way.
That is of giving the option to pass to the actor a list of exceptions for which:
This is actually the equivalent of https://docs.celeryproject.org/en/stable/userguide/tasks.html#Task.throws
This is very fitting for use cases where tasks use parameters or data provided by users, for combinations of whom the task may fail in expected ways (that you can't know itwill happen before running it).
In such cases there is no way this may succeed by retrying, there is no value in polluting your sentry/logs with error level info, and you want the caller side to be able to get the resulting error of this task so it can return the appropriate error response.
This also applies to other types of expected errors.
@Bogdanp whould you integrate this feature? I can work on the implementation and provide a PR
The text was updated successfully, but these errors were encountered: