Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
4c0b6be tocfdf9ceCompareGetting the actual message count might be more complicated:#39127. |
alexander-schranz commentedMay 2, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@chalasr yes |
src/Symfony/Component/Messenger/Bridge/Redis/Transport/Connection.php OutdatedShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
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 |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
a403e97 toffd2b07Compareffd2b07 toe1a896fCompareThank 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.