Skip to content

Conversation

shohamazon
Copy link
Collaborator

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@shohamazon shohamazon requested a review from a team as a code owner February 15, 2024 15:33
@shohamazon shohamazon force-pushed the python/sismember branch 2 times, most recently from 31a61f6 to 7b647dd Compare February 15, 2024 15:48
@shohamazon shohamazon requested a review from barshaul February 15, 2024 16:00
@shohamazon shohamazon added the python 🐍 Python wrapper label Feb 15, 2024
member (str): The member to check for existence in the set.

Returns:
int: 1 if the member exists in the set, 0 otherwise.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as in the HSETNX PR - can we parse this command response in rust and return a boolean instead of an int?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will


Returns:
int: 1 if the member exists in the set, 0 otherwise.
If `key` is not a set, it is treated an an empty set and the command returns 0.
Copy link
Collaborator

@barshaul barshaul Feb 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If key is not a set, it is treated an an empty => If key is not a set, it is treated as an empty s

@shohamazon shohamazon merged commit 3809a28 into valkey-io:main Feb 29, 2024
@shohamazon shohamazon deleted the python/sismember branch February 29, 2024 13:23
cyip10 pushed a commit to Bit-Quill/valkey-glide that referenced this pull request Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
python 🐍 Python wrapper
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants