-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[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:retry
command, Messenger displays the number of messages to retry.Actually, this count is not displayed when using
RedisTransport
. This PR adds this functionality.