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.

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

BooleanOutput set function is incorrect #1553

Closed
paladin4409 opened this issue Dec 8, 2020 · 1 comment
Closed

BooleanOutput set function is incorrect #1553

paladin4409 opened this issue Dec 8, 2020 · 1 comment
Labels
status: duplicate A duplicate of another issue

Comments

@paladin4409
Copy link

Bug Report

Current Behavior

use xgroup delconsumer command to delete a consumer for redis stream, the result returns pending messages count.
but invoke LettuceStreamCommands xGroupDelConsumer, return BooleanOutput object, the BooleanOutput set output function:

public void set(long integer) {
output = (integer == 1) ? Boolean.TRUE : Boolean.FALSE;
}

so, if the pending messages is more than one, this BooleanOutput set Boolean.FALSE! but it is actually success.

Environment

  • Lettuce version(s): [5.3.0.RELEASE]
  • Redis version: [5.0.0]
@paladin4409 paladin4409 added the type: bug A general bug label Dec 8, 2020
@paladin4409 paladin4409 changed the title BooleanOutput set funcation is incorrect BooleanOutput set function is incorrect Dec 8, 2020
@mp911de mp911de added status: duplicate A duplicate of another issue and removed type: bug A general bug labels Dec 8, 2020
@mp911de
Copy link
Collaborator

mp911de commented Dec 8, 2020

That was fixed with #1377 for version 6.0

@mp911de mp911de closed this as completed Dec 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: duplicate A duplicate of another issue
Projects
None yet
Development

No branches or pull requests

2 participants