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

Add GEOSEARCH to read-only commands #2568 #2569

Closed

Conversation

kajte
Copy link
Contributor

@kajte kajte commented Dec 12, 2023

Solves #2568

Set GEOSEARCH as read-only command to allow running the command on replicas. Works similar to #2198.

Set GEOSEARCH as read-only command to allow running the command on replicas.
@mp911de mp911de added status: help-wanted An issue that a contributor can help us with status: good-first-issue An issue that can only be worked on by brand new contributors and removed status: help-wanted An issue that a contributor can help us with labels Dec 13, 2023
@thx01138
Copy link

@mp911de @dengliming Can this PR be merged? I want to add another PR just like this to support FCALL_RO but since the unit test for this counts number of commands in the read only list, I want to open my PR after this is merged to main.

@thx01138
Copy link

@mp911de Is this ready to merge? Would you do the honors?

@mp911de mp911de added this to the 6.3.1.RELEASE milestone Jan 3, 2024
@mp911de mp911de added type: feature A new feature and removed status: good-first-issue An issue that can only be worked on by brand new contributors labels Jan 3, 2024
@mp911de mp911de self-assigned this Jan 3, 2024
@mp911de mp911de added type: bug A general bug and removed type: feature A new feature labels Jan 3, 2024
mp911de pushed a commit that referenced this pull request Jan 3, 2024
Set GEOSEARCH as read-only command to allow running the command on replicas.

Original pull request: #2569
mp911de added a commit that referenced this pull request Jan 3, 2024
mp911de pushed a commit that referenced this pull request Jan 3, 2024
Set GEOSEARCH as read-only command to allow running the command on replicas.

Original pull request: #2569
mp911de added a commit that referenced this pull request Jan 3, 2024
@mp911de
Copy link
Collaborator

mp911de commented Jan 3, 2024

Thank you for your contribution. That's merged, polished, and backported now.

@mp911de mp911de closed this Jan 3, 2024
@mp911de mp911de linked an issue Jan 3, 2024 that may be closed by this pull request
@kajte
Copy link
Contributor Author

kajte commented Jan 3, 2024

Thank you for your contribution. That's merged, polished, and backported now.

Awesome! Thanks to you too!

@tishun tishun mentioned this pull request Jun 6, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Geosearch and FCALL_RO commands go to the master node
3 participants