Skip to content

Add DAGRUN_RO command #334

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Merged
merged 2 commits into from
Apr 22, 2020
Merged

Add DAGRUN_RO command #334

merged 2 commits into from
Apr 22, 2020

Conversation

lantiga
Copy link
Contributor

@lantiga lantiga commented Apr 22, 2020

This PR addresses #333

DAGRUNRO is a DAGRUN command that has no support for PERSIST. This means that one can run DAGRUNRO against replicas in a master-replica configuration.

#330 will need to add this command to the docs

@lantiga lantiga requested a review from filipecosta90 April 22, 2020 10:15
filipecosta90
filipecosta90 approved these changes Apr 22, 2020
View reviewed changes
Copy link
Collaborator

@filipecosta90 filipecosta90 left a comment

Choose a reason for hiding this comment

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

LGTM @lantiga . Lets do the final CI check and merge

@filipecosta90 filipecosta90 merged commit 58cfc3d into master Apr 22, 2020
@filipecosta90 filipecosta90 changed the title Add DAGRUNRO command Add DAGRUN_RO command May 1, 2020
@filipecosta90 filipecosta90 deleted the dagro branch May 1, 2020 10:31
lantiga pushed a commit that referenced this pull request May 6, 2020
@filipecosta90
6a520c9 
 
Merge pull request #334 from RedisAI/dagro

Add DAGRUNRO command
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@filipecosta90 filipecosta90 filipecosta90 approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lantiga @filipecosta90