Skip to content

fix: when refreshSlotsCache is called concurrently, call the callback only when the refresh process is done #1881

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.

Sign up for GitHub

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

Jump to bottom
Merged
leibale merged 1 commit into from
Apr 16, 2024
Merged

fix: when refreshSlotsCache is called concurrently, call the callback only when the refresh process is done #1881

leibale merged 1 commit into from
Apr 16, 2024

Conversation

@leibale
Copy link
Contributor

@leibale leibale commented Apr 16, 2024

No description provided.

@leibale
fix: when refreshSlotsCache is callback concurrently, call the call…
26cc9e5 
…back only when the refresh process is done
@leibale leibale merged commit 804ee07 into redis:main Apr 16, 2024
github-actions bot pushed a commit that referenced this pull request Apr 16, 2024
@ioredis-robot @semantic-release-bot
chore(release): 5.4.0 [skip ci]
4f2ab44 
# [5.4.0](v5.3.2...v5.4.0) (2024-04-16)

### Bug Fixes

* when `refreshSlotsCache` is callback concurrently, call the callback only when the refresh process is done ([#1881](#1881)) ([804ee07](804ee07))

### Features

* add support for `socketTimeout` in `Redis` ([#1882](#1882)) ([673ac77](673ac77))
@github-actions
Copy link

github-actions bot commented Apr 16, 2024

🎉 This PR is included in version 5.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@leibale leibale changed the title fix: when refreshSlotsCache is callback concurrently, call the callback only when the refresh process is done fix: when refreshSlotsCache is called concurrently, call the callback only when the refresh process is done Apr 16, 2024
@olokotoh olokotoh mentioned this pull request May 20, 2024
Open
@officialmofabs officialmofabs mentioned this pull request May 26, 2024
Open
@confused-moniker confused-moniker mentioned this pull request May 26, 2024
Closed
@baderdean baderdean mentioned this pull request Jun 12, 2024
Closed
@dev-trilobyte dev-trilobyte mentioned this pull request Jul 3, 2024
Merged
@Stanislav1975 Stanislav1975 mentioned this pull request Aug 9, 2024
Open
@WesleyR10 WesleyR10 mentioned this pull request Aug 21, 2024
Open
@vs4vijay vs4vijay mentioned this pull request Sep 10, 2024
Closed
@juls0730 juls0730 mentioned this pull request Sep 30, 2024
Closed
@htemelski htemelski mentioned this pull request Apr 8, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@leibale