Add missing stream key prefixes #1230
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jessarcher
changed the title
tillkruss
requested changes
Mar 28, 2023
tillkruss
approved these changes
Mar 29, 2023
|
@jessarcher Do you need a release? |
|
@tillkruss Not urgently, but it would be appreciated when you have the time. |
|
@vladvildanov Can you add these commands to |
|
@tillkruss Did it within my PR 👌 |
|
Thanks @vladvildanov ❤️🔥 |
vladvildanov
added a commit
that referenced
this pull request
Apr 3, 2023
* Added support for new arguments for BITPOS, BITCOUNT commands (#1045) * Added support for new arguments for EXPIRE, EXPIREAT commands (#1046) * Extended core support by implementing SORT_RO command (#1044) * Added support for SORT_RO command * Codestyle fixes * Added command description --------- Co-authored-by: Vladyslav Vildanov <vladyslavvildanov@Vladyslav-Vildanov-MacBook-Pro.local> * fix deprecated call * Added support for container commands (#1049) * 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 <vladyslavvildanov@Vladyslav-Vildanov-MacBook-Pro.local> * Added stream commands to KeyPrefixProcessor (#1051) Co-authored-by: Vladyslav Vildanov <vladyslavvildanov@Vladyslav-Vildanov-MacBook-Pro.local> * Fix return type of ReplicationInterface::getSlaves (#1111) * Codestyle fixes * Changed return annotation * Add option for disabling reads to slaves in replication topologies (#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 <tillkruss@users.noreply.github.com> * match `MasterSlaveReplication` (#1175) * match `MasterSlaveReplication` also the name of this method 😬 * Update SentinelReplication.php * Added CODEOWNERS file (#1181) * 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 <vladyslavvildanov@Vladyslav-Vildanov-MacBook-Pro.local> Co-authored-by: Till Krüss <tillkruss@users.noreply.github.com> * Deprecate webdis and phpiredis (#1179) * Deprecate webdis and phpiredis * remove space * tag v2.1.2 * Extended core support by implementing FCALL_RO command (#1191) * typo * Extended core support by implementing ACL SETUSER, GETUSER, DRYRUN (#1193) * Added support for ACL GETUSER, SETUSER, DRYRUN commands * Change test to support Redis > 6.0.0 * Removed selectors check * Add missing stream key prefixes (#1230) * Added missing stream key prefixes * Tested stream command key prefixing * Codestyle fix * Changed idx name for different index types --------- Co-authored-by: Vladyslav Vildanov <vladyslavvildanov@Vladyslav-Vildanov-MacBook-Pro.local> Co-authored-by: Till Krüss <till@kruss.io> Co-authored-by: Stephan <glaubinix@users.noreply.github.com> Co-authored-by: David Bomba <turbo124@gmail.com> Co-authored-by: Till Krüss <tillkruss@users.noreply.github.com> Co-authored-by: Jess Archer <jess@jessarcher.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#1051 fixed an issue where Predis was not applying key prefixes to stream commands, however, it missed a couple of commands that are otherwise supported by Predis.
This PR adds key prefix support for
XREVRANGEandXTRIM. There are additional Redis stream commands such asXREADthat are missing from this list, but they don't have a corresponding class insrc/Command/Redis/so I assume they are unsupported in Predis at this time.