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

feat(raiko): make redis able to re-connect #432

Merged
merged 4 commits into from
Dec 12, 2024
Merged

feat(raiko): make redis able to re-connect #432

merged 4 commits into from
Dec 12, 2024

Conversation

smtmfft
Copy link
Contributor

@smtmfft smtmfft commented Dec 12, 2024

in k8s development, the redis sometimes restarts without awareness, add reconnect mechanism to avoid fatal errors.

Copy link
Contributor

@keroro520 keroro520 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
Left 2 suggestions.

taskdb/src/redis_db.rs Outdated Show resolved Hide resolved
taskdb/src/redis_db.rs Outdated Show resolved Hide resolved
@smtmfft smtmfft added this pull request to the merge queue Dec 12, 2024
Merged via the queue into main with commit e530f4f Dec 12, 2024
3 checks passed
@smtmfft smtmfft deleted the fix-redis-reconn branch December 12, 2024 05:16
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.

3 participants