Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.2k
Commitce2b086
committed
minor#14888 [Messenger] More on unique Redis consumer names (danmichaelo)
This PR was merged into the 5.2 branch.Discussion----------[Messenger] More on unique Redis consumer namesI managed to miss the warning that the (strem,group,consumer) combination must be unique. I think it would make it less easy to miss if it was also mentioned in the Supervisor section, so I've tried to add a mention of it there. I also expanded the original warning block a bit.There are many different combinations of transports and ways to manage workers, so I understand that the docs should not delve too deep into each one, but I think it's worthwhile to cover the Redis + Supervisor combo quite well, since it's one of the easier combinations to get started with, which also can work really well. Let me know what you think.Related:#11869 #35358Commits-------e73e923 [Messenger] More on unique Redis consumer names1 file changed
+19
-4
lines changedLines changed: 19 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
626 | 626 |
| |
627 | 627 |
| |
628 | 628 |
| |
629 |
| - | |
| 629 | + | |
| 630 | + | |
| 631 | + | |
| 632 | + | |
| 633 | + | |
| 634 | + | |
| 635 | + | |
| 636 | + | |
| 637 | + | |
| 638 | + | |
| 639 | + | |
| 640 | + | |
630 | 641 |
| |
631 | 642 |
| |
632 | 643 |
| |
| |||
1239 | 1250 |
| |
1240 | 1251 |
| |
1241 | 1252 |
| |
1242 |
| - | |
1243 |
| - | |
1244 |
| - | |
| 1253 | + | |
| 1254 | + | |
| 1255 | + | |
| 1256 | + | |
| 1257 | + | |
| 1258 | + | |
| 1259 | + | |
1245 | 1260 |
| |
1246 | 1261 |
| |
1247 | 1262 |
| |
|
0 commit comments
Comments
(0)