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

randomkey return V not K #1240

Closed
honsunrise opened this issue Mar 12, 2020 · 1 comment
Closed

randomkey return V not K #1240

honsunrise opened this issue Mar 12, 2020 · 1 comment
Labels
type: bug A general bug
Milestone
5.3.0

Comments

@honsunrise
Copy link

honsunrise commented Mar 12, 2020 •
edited
Loading

https://github.com/lettuce-io/lettuce-core/blob/efb0b6c49b5a9b66bc4bd88c5961d64cb9d52556/src/main/java/io/lettuce/core/api/reactive/RedisKeyReactiveCommands.java#L241

Why randomkey return K instead of V?
@honsunrise honsunrise added the type: bug A general bug label Mar 12, 2020
@mp911de mp911de added this to the 5.3.0 milestone Mar 12, 2020
@mp911de
Copy link
Collaborator

mp911de commented Mar 12, 2020

That's a bug we need to fix.

@mp911de mp911de changed the title randomkey return V not K. randomkey return V not K Mar 13, 2020
mp911de added a commit that referenced this issue Mar 13, 2020
@mp911de
Fix return type of randomkey #1240
d8ef525 
The return type of randomkey now uses the key type. Previously the return value was declared as value type.
mp911de added a commit that referenced this issue Mar 13, 2020
@mp911de
Fix return type of randomkey #1240
a253019 
The return type of randomkey now uses the key type. Previously the return value was declared as value type.
@mp911de mp911de closed this as completed Mar 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: bug A general bug
Projects
None yet
Milestone
5.3.0
Development

No branches or pull requests
2 participants
@mp911de @honsunrise