Skip to content
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.

Sign up for GitHub

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

Jump to bottom

XGROUP DELCONSUMER should return pending message count #1377

Closed
mp911de opened this issue Aug 3, 2020 · 1 comment
Closed

XGROUP DELCONSUMER should return pending message count #1377

mp911de opened this issue Aug 3, 2020 · 1 comment
Labels
type: breaking Breaking change type: bug A general bug
Milestone
6.0 RC1

Comments

@mp911de
Copy link
Collaborator

mp911de commented Aug 3, 2020

xgroupDelconsumer returns a boolean flag instead of the pending message count.
@mp911de mp911de added the type: bug A general bug label Aug 3, 2020
@mp911de mp911de added this to the 6.0 RC1 milestone Aug 3, 2020
mp911de added a commit that referenced this issue Aug 3, 2020
@mp911de
Stream commands xgroupDelconsumer(…) now returns pending count #1377
13499bc 
xgroupDelconsumer now returns the number of pending messages. Previously, it returned just a boolean flag.
@mp911de mp911de added the type: breaking Breaking change label Aug 3, 2020
@mp911de mp911de closed this as completed Aug 3, 2020
@momomo
Copy link

momomo commented Aug 27, 2020

Should reassign the pending ones similar to what is proposed under XCLAIM https://redis.io/commands/xpending

@mp911de mp911de mentioned this issue Dec 8, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: breaking Breaking change type: bug A general bug
Projects
None yet
Milestone
6.0 RC1
Development

No branches or pull requests
2 participants
@mp911de @momomo