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

Python: Add ZSCAN and HSCAN commands #1732

Merged
merged 19 commits into from
Jul 1, 2024

Conversation

GumpacG
Copy link
Collaborator

@GumpacG GumpacG commented Jun 30, 2024

Issue #, if available:

Description of changes:
https://valkey.io/commands/zscan/
https://valkey.io/commands/hscan/

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

@GumpacG GumpacG requested a review from a team as a code owner June 30, 2024 16:34
Copy link
Collaborator

@Yury-Fridlyand Yury-Fridlyand left a comment

Choose a reason for hiding this comment

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

Please double check that you don't have copy-paste artifacts

.github/workflows/python.yml Outdated Show resolved Hide resolved
python/python/glide/async_commands/core.py Outdated Show resolved Hide resolved
python/python/glide/async_commands/core.py Outdated Show resolved Hide resolved
python/python/glide/async_commands/transaction.py Outdated Show resolved Hide resolved
python/python/tests/test_async_client.py Show resolved Hide resolved
@GumpacG GumpacG force-pushed the python/guiang_zscan branch from d47eb43 to dcfe43d Compare July 1, 2024 00:11
@Yury-Fridlyand Yury-Fridlyand added the python Python wrapper label Jul 1, 2024
python/python/glide/async_commands/core.py Show resolved Hide resolved
python/python/tests/test_async_client.py Outdated Show resolved Hide resolved
@GumpacG GumpacG force-pushed the python/guiang_zscan branch from d038ba9 to 205eab7 Compare July 1, 2024 00:51
@acarbonetto acarbonetto merged commit 397c641 into valkey-io:main Jul 1, 2024
8 checks passed
@acarbonetto acarbonetto deleted the python/guiang_zscan branch July 1, 2024 02:31
jamesx-improving pushed a commit to Bit-Quill/valkey-glide that referenced this pull request Jul 1, 2024
* Added zscan command

* Added HSCAN command

* Fix tests

* Fix tests

* Fix tests

* Fix tests

* Fix tests

* Fix tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Fixed tests and addressed comments

* Update python/python/tests/test_async_client.py

Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com>

---------

Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com>
acarbonetto pushed a commit that referenced this pull request Jul 1, 2024
* Python: add WATCH and UNWATCH commands

* isort

* update redis_client type for test_unwatch_with_route()

* add watch to the correct cluster test

* add cluster mode note

* address comments from GumpacG

* Python: added XAUTOCLAIM command (#1718)

* Python: add XAUTOCLAIM command

* minor doc update

* Minor doc update

* PR suggestions

* Update test assertions with string conversions

* PR suggestions

* Add clarifying comments

* Python: Add ZSCAN and HSCAN commands (#1732)

* Added zscan command

* Added HSCAN command

* Fix tests

* Fix tests

* Fix tests

* Fix tests

* Fix tests

* Fix tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Fixed tests and addressed comments

* Update python/python/tests/test_async_client.py

Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com>

---------

Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com>

* Python: add WATCH and UNWATCH commands

* Python: add WATCH and UNWATCH commands

* add changelog

* fix broken syntax after solving conflicts

---------

Co-authored-by: Aaron <69273634+aaron-congo@users.noreply.github.com>
Co-authored-by: Guian Gumpac <guian.gumpac@improving.com>
Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com>
cyip10 pushed a commit to Bit-Quill/valkey-glide that referenced this pull request Jul 16, 2024
* Added zscan command

* Added HSCAN command

* Fix tests

* Fix tests

* Fix tests

* Fix tests

* Fix tests

* Fix tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Fixed tests and addressed comments

* Update python/python/tests/test_async_client.py

Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com>

---------

Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com>
cyip10 pushed a commit to Bit-Quill/valkey-glide that referenced this pull request Jul 16, 2024
* Python: add WATCH and UNWATCH commands

* isort

* update redis_client type for test_unwatch_with_route()

* add watch to the correct cluster test

* add cluster mode note

* address comments from GumpacG

* Python: added XAUTOCLAIM command (valkey-io#1718)

* Python: add XAUTOCLAIM command

* minor doc update

* Minor doc update

* PR suggestions

* Update test assertions with string conversions

* PR suggestions

* Add clarifying comments

* Python: Add ZSCAN and HSCAN commands (valkey-io#1732)

* Added zscan command

* Added HSCAN command

* Fix tests

* Fix tests

* Fix tests

* Fix tests

* Fix tests

* Fix tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Debug tests

* Fixed tests and addressed comments

* Update python/python/tests/test_async_client.py

Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com>

---------

Co-authored-by: Yury-Fridlyand <yury.fridlyand@improving.com>

* Python: add WATCH and UNWATCH commands

* Python: add WATCH and UNWATCH commands

* add changelog

* fix broken syntax after solving conflicts

---------

Co-authored-by: Aaron <69273634+aaron-congo@users.noreply.github.com>
Co-authored-by: Guian Gumpac <guian.gumpac@improving.com>
Co-authored-by: Yury-Fridlyand <yury.fridlyand@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