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

Azure SQL transient error #182

Open
conreaux opened this issue Aug 11, 2023 · 3 comments
Open

Azure SQL transient error #182

conreaux opened this issue Aug 11, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@conreaux
Copy link

conreaux commented Aug 11, 2023

We occasionally encounter a transient Azure SQL error 47073 (see definition below) in our DF application. Our IT department is investigating root cause. It can hit any DF in our app. Is this a candidate to add to the transient errors list for automatic retry? Could additional error codes be configurable?

We could add custom retry handlers to activity and sub-orchestrator functions, but that doesn't address the other cases (entities, external events, etc). Other suggestions for handling such an error (other than wrapping every DF call in a try/catch)?

Reason: Unable to set Deny Public Network Access to Yes since there is no private endpoint enabled to access the server. Please set up private endpoints and retry the operation (https://learn.microsoft.com/azure/sql-database/sql-database-private-endpoint-overview#how-to-set-up-private-link-for-azure-sql-database).

@cgillum
Copy link
Member

cgillum commented Aug 29, 2023

@conreaux the description of 47073 sounds like it's not supposed to be transient, but you're observing it happening transiently? Do I understand that correctly?

@cgillum cgillum self-assigned this Aug 29, 2023
@cgillum cgillum added Needs: Author Feedback Awaiting feedback from the issue author. and removed Needs: Triage 🔍 labels Aug 29, 2023
@conreaux
Copy link
Author

Yes, I agree that it should not be transient. However, we encounter this error intermittently and typically for a duration of 3-5 minutes. Our IT and Microsoft support are investigating, but it does seem related to DNS resolution somehow defaulting to the public endpoint even though a private endpoint is configured on the Azure SQL database (our app is running in an isolated App Service Environment). Similar issues are mentioned in this Reddit post. I understand that it probably does not fit into the transient errors list and probably not much that can be done in the MSSQL provider, but it does cause problems since any DF call can fail. Maybe just looking for error handling advice to make our app fault tolerant in cases where database errors may escape the retry logic.

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs: Attention 👋 Issue needs attention from maintainers and removed Needs: Author Feedback Awaiting feedback from the issue author. labels Aug 30, 2023
@cgillum
Copy link
Member

cgillum commented Aug 30, 2023

I think your suggestion of allowing custom status codes to be configured makes sense for a case like this. I’m supportive of this idea.

@cgillum cgillum added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed and removed Needs: Attention 👋 Issue needs attention from maintainers labels Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants