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 FUNCTION LOAD command #1699

Merged

Conversation

yipin-chen
Copy link
Collaborator

Issue #, if available:
N/A

Description of changes:
https://redis.io/docs/latest/commands/function-load/

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

@yipin-chen yipin-chen added the python Python wrapper label Jun 27, 2024
@yipin-chen yipin-chen requested a review from a team as a code owner June 27, 2024 19:00
@shohamazon
Copy link
Collaborator

You can use the opened pr and work on it :)

@yipin-chen yipin-chen force-pushed the python/integ_yipin_function-load branch from 4d4b1dc to 152656b Compare June 27, 2024 19:28
@yipin-chen
Copy link
Collaborator Author

You can use the opened pr and work on it :)

I don't have the access to your opened PR. After discussing with Andrew and Yury, we decided to create a new PR. I will reference your PR.

@yipin-chen
Copy link
Collaborator Author

This new Python FUNCTION LOAD PR is based on #1589.
I will close PR 1589 after this PR is merged.
I only addressed review comments and merged with the latest changes.

@acarbonetto acarbonetto merged commit 187b2ba into valkey-io:main Jun 27, 2024
6 checks passed
@acarbonetto acarbonetto deleted the python/integ_yipin_function-load branch June 27, 2024 22:03
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.

4 participants