Skip to content

Python: add ZREVRANK command - #1614

Merged
acarbonetto merged 3 commits into
valkey-io:mainfrom
Bit-Quill:python/integ_acongo_zrevrank
Jun 20, 2024
Merged

acarbonetto merged 3 commits into
valkey-io:mainfrom
Bit-Quill:python/integ_acongo_zrevrank

Conversation

@aaron-congo

@aaron-congo aaron-congo commented Jun 19, 2024

Copy link
Copy Markdown
Contributor

Issue #, if available:
N/A

Description of changes:
https://redis.io/docs/latest/commands/zrevrank/

  • Returns the rank of member in the sorted set stored at key, where scores are ordered from the highest to lowest, starting from 0
  • Policies: none (see here)

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

@aaron-congo
aaron-congo requested a review from a team as a code owner June 19, 2024 23:32
@aaron-congo aaron-congo added the python 🐍 Python wrapper label Jun 19, 2024

@acarbonetto acarbonetto left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

Comment thread python/python/glide/async_commands/core.py Outdated
@acarbonetto
acarbonetto merged commit 499e2cf into valkey-io:main Jun 20, 2024
@acarbonetto
acarbonetto deleted the python/integ_acongo_zrevrank branch June 20, 2024 03:22
cyip10 pushed a commit to Bit-Quill/valkey-glide that referenced this pull request Jun 24, 2024
* Python: add ZREVRANK command

* Relocate zrevrank to be adjacent to zrank

---------

Co-authored-by: Andrew Carbonetto <andrew.carbonetto@improving.com>
affonsov pushed a commit that referenced this pull request Aug 21, 2026
* Python: add ZREVRANK command

* Relocate zrevrank to be adjacent to zrank

---------

Co-authored-by: Andrew Carbonetto <andrew.carbonetto@improving.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python 🐍 Python wrapper

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants