-
Notifications
You must be signed in to change notification settings - Fork 275
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
Configurable retry - questions/clarifications #993
Comments
Just a reminder about implementing SqlException.IsTransient, which is related to this. |
@ErikEJ that list has been validated with multiple teams within AzureSQL team (e.g. Gateway, resource governance, etc.) and as of now represents the most accurate representation of most common transient errors for the service. The list on EFCore seems a superset that includes also some transaction related errors that may be specific to their business logic, but we couldn't find a good use case to treat them as default. Of course users can still add those error codes in config file or in code if they will find them useful. |
Thanks for the info, @scoriani @AndriySvyryd @ajcvickers FYI! |
@scoriani So it is currently not possible to remove any? |
Not for now. To be completely honest, that feature was in my initial prototype 😀. I was using + and - signs after the error numbers in the list to indicate error codes to add or remove from the default list, but during internal reviews that was considered a bit too cumbersome. |
Also overkill, given the limited size of the list! |
1: Is it possible to remove entries from the default list:
SqlClient/src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/Reliability/SqlConfigurableRetryFactory.cs
Line 41 in c37092a
2: Why is the default list different from https://github.com/dotnet/efcore/blob/main/src/EFCore.SqlServer/Storage/Internal/SqlServerTransientExceptionDetector.cs#L24 (there may be good reasons...)
@DavidEshtehari
The text was updated successfully, but these errors were encountered: