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

Auto retry potential DB transient failures with tenacity #83

Closed
viniarck opened this issue Mar 30, 2022 · 0 comments · Fixed by #90
Closed

Auto retry potential DB transient failures with tenacity #83

viniarck opened this issue Mar 30, 2022 · 0 comments · Fixed by #90
Assignees
Labels
enhancement New feature or request epic_nosql_db_foundation NoSQL Database Foundation

Comments

@viniarck
Copy link
Member

As highlighted on pymongo docs, it's responsibility of the application to handle certain exceptions like AutoReconnect when the driver can't auto retry. So, this issue is to add support for retry capabilities with tenacity for certain writes and reads to make the integration production-grade. If you have suggestions to consider other retry library let me know, tenacity is great, I've used it in production before.

What's being proposed here also goes hand in hand with the dead letter dict structure, mapped on this issue, if all retries fail it'll land there.

Side note: There's also use case for tenacity on other NApps as mapped here, I think it makes sense to ship it on kytos core when we get the chance in the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request epic_nosql_db_foundation NoSQL Database Foundation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant