-
-
Notifications
You must be signed in to change notification settings - Fork 989
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
Resolve merge conflicts between v2.x and main #1247
Conversation
* Added support for SORT_RO command * Codestyle fixes * Added command description --------- Co-authored-by: Vladyslav Vildanov <[email protected]>
* Added support for container commands FUNCTION LOAD, FUNCTION DELETE and FCALL * Changed ContainerInterface and AbstractContainer * Re-implement logic of abstract methods --------- Co-authored-by: Vladyslav Vildanov <[email protected]>
Co-authored-by: Vladyslav Vildanov <[email protected]>
…redis#1168) * Add option for disabling reads to slaves in replication topologies (cherry picked from commit e132c31) * remove empty line --------- Co-authored-by: Till Krüss <[email protected]>
* match `MasterSlaveReplication` also the name of this method 😬 * Update SentinelReplication.php
* Added CODEOWNERS file * Update CODEOWNERS * Applied whole Redis commands path as @vladvildanov codeowner * Set @predis/team as Redis commands codeowners * Update CODEOWNERS --------- Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]>
* Deprecate webdis and phpiredis * remove space
…redis#1193) * Added support for ACL GETUSER, SETUSER, DRYRUN commands * Change test to support Redis > 6.0.0 * Removed selectors check
* Added missing stream key prefixes * Tested stream command key prefixing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's wait on @tillkruss to merge obvi
@chayim It's not a main PR, it's just resolving conflicts between |
@shacharPash