-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Redis stream - use streams XLEN instead of XPENDING #4277
Labels
Comments
jkbmdk
added
feature-request
All issues for new features that have not been committed to
needs-discussion
labels
Feb 24, 2023
Hello, |
Sure, should the solution be implemented for redis streams scaler as an additional configuration parameter and when it would be set there would be no need to set pendingEntriesCount or consumerGroup? |
I think so, you can make them optional and raise an error in case of they are required and they are missing (and properly document it xD) |
jkbmdk
pushed a commit
to jkbmdk/keda
that referenced
this issue
Mar 20, 2023
jkbmdk
pushed a commit
to jkbmdk/keda
that referenced
this issue
Mar 20, 2023
Signed-off-by: jmadajczak <jmadajczak@transparentdata.pl>
2 tasks
jkbmdk
pushed a commit
to jkbmdk/keda
that referenced
this issue
Mar 20, 2023
Signed-off-by: jmadajczak <jmadajczak@transparentdata.pl>
7 tasks
jkbmdk
pushed a commit
to jkbmdk/keda
that referenced
this issue
Mar 20, 2023
Signed-off-by: jmadajczak <jmadajczak@transparentdata.pl>
jkbmdk
pushed a commit
to jkbmdk/keda
that referenced
this issue
Mar 20, 2023
Signed-off-by: jmadajczak <jmadajczak@transparentdata.pl>
1 task
JorTurFer
added a commit
that referenced
this issue
Mar 28, 2023
7 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Proposal
When it comes to using redis streams as a message queue, as php symfony messenger does, declaring the XPENDING parameter as a scaler attribute will not be useful in some scenarios, because it usually returns the current number of subscribers.
The solution I would propose is to make the redis stream scaler optionally dependent on the XLEN parameter.
Use-Case
First of all we are using php symfony messenger and redis. The only way to combine those two is by using redis streams. In our case, we publish many messages once a day and much less throughout the day. Our goal is to quickly subscribe messages from the initial feed (say, within a few hours), and then maintain enough subscribers to handle incoming messages.
Is this a feature you are interested in implementing yourself?
Maybe
Anything else?
No response
The text was updated successfully, but these errors were encountered: