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

Allow retrying on Service Unavailable errors from OpenAI #1407

Merged
merged 1 commit into from
Jul 22, 2023

Conversation

@MaartenGr
Copy link
Owner

Thank you for the PR! I just read through the links you provided and it seems that they did not specifically mention that this error could be directly resolved with backoff. Just to be sure, is the unreliability that you refer to something that should be fixed with backoff or is it something that temporarily happens within their servers? If it is the latter, then it might make unnecessary API calls as it will be fixed whenever their servers are not overloaded anymore.

@agamble
Copy link
Contributor Author

agamble commented Jul 13, 2023

Hey Maarten, thanks for the comment.

You're right that the links I shared don't specify that the errors are resolved via backoff, my intent in sharing them was to corroborate that many others are experiencing reliability issues with OpenAI - specifically with intermittent ServiceUnavailable errors. The links do say that these errors are due to OpenAI receiving too much traffic though, so a retry with backoff makes sense to me from that PoV.

Anecdotally, this change has completely resolve reliability issues I was seeing with using OpenAI during the representation stage. Prior to this change, it was extremely common to receive a ServiceUnavailable error from OpenAI, which would halt my entire clustering workflow.

@MaartenGr
Copy link
Owner

Apologies for the late reply!

Anecdotally, this change has completely resolve reliability issues I was seeing with using OpenAI during the representation stage. Prior to this change, it was extremely common to receive a ServiceUnavailable error from OpenAI, which would halt my entire clustering workflow.

Totally fair. Since your message/PR, there were also more users with this problem, so it definitely makes sense. When the pipeline runs, I'll make sure to merge this PR.

@MaartenGr MaartenGr merged commit df862d3 into MaartenGr:master Jul 22, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants