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
Weaviate: Add retry #32038
Weaviate: Add retry #32038
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
Before Merging a Connector Pull RequestWow! What a great pull request you have here! 🎉 To merge this PR, ensure the following has been done/considered for each connector added or updated:
If the checklist is complete, but the CI check is failing,
|
This reverts commit ba78485.
…te into flash1293/weaviate-retry
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Thanks! 👍 🚢
Fixes #31991
The Weaviate client is already retrying requests automatically in situations like timeouts. However, some situations like in the linked issue are not handled by this default setting.
This PR is enabling retries for all errors on the Weaviate client to improve this issue.