-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
[Messenger] Make Redis messages countable #46229
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
[Messenger] Make Redis messages countable #46229
Conversation
4c0b6be to
cfdf9ce
Compare
|
Getting the actual message count might be more complicated: #39127. |
|
@chalasr yes |
src/Symfony/Component/Messenger/Bridge/Redis/Transport/Connection.php
Outdated
Show resolved
Hide resolved
|
Thanks @alexander-schranz ! I updated my code. But, I am wondering if we have to limit |
|
I did have a look at the latest changes about redis. The new version Redis 7 provides redis/redis#9127 Not 100% sure but could be that lag provide the information we need. |
|
I ran tests on a Redis 7.x a |
a403e97 to
ffd2b07
Compare
ffd2b07 to
e1a896f
Compare
|
Thank you @Jean-Beru. |
|
Good work @Jean-Beru 👍 |
|
Thanks a lot for your help @alexander-schranz ! |
Each Messenger Transport can return the number of messages present in the queue. For example, when we use the
messenger:failed:retrycommand, Messenger displays the number of messages to retry.Actually, this count is not displayed when using
RedisTransport. This PR adds this functionality.