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

Java: Add Zmscore command. (Sorted Set Command Group) #173

Merged

Conversation

SanHalacogluImproving
Copy link

No description provided.

@@ -34,6 +34,7 @@
* Core: Enabled Cluster Mode periodic checks by default ([#1089](https://github.com/aws/glide-for-redis/pull/1089))
* Node: Added Rename command. ([#1124](https://github.com/aws/glide-for-redis/pull/1124))
* Python: Added JSON.TOGGLE command ([#1184](https://github.com/aws/glide-for-redis/pull/1184))
* Java: Added ZMSCORE command (TODO ADD HERE PR #)

Choose a reason for hiding this comment

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

No need to mention java here, but maybe core. awaiting for AWS team to reply whether we need to record that here or not.

@@ -135,6 +135,7 @@ enum RequestType {
Rename = 91;
DBSize = 92;
Brpop = 93;
ZMScore = 94;

Choose a reason for hiding this comment

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

94 is already occupied by a PR on upstream => you will have a conflict
Pick another number. 99 is the next free AFAIK

@SanHalacogluImproving SanHalacogluImproving merged commit 73cd123 into java/integ_SanH_add_Zmscore Apr 4, 2024
39 of 43 checks passed
acarbonetto pushed a commit that referenced this pull request Apr 10, 2024
* Java: Add Zmscore command. (Sorted Set Command Group) (#173)

* Minor update + remove change log.

* Minor documentation update.

* Minor IT update.

* Rebase + Spotless.

* Minor IT test update.
alex-arzola-imp pushed a commit that referenced this pull request Apr 12, 2024
* Java: Add Zmscore command. (Sorted Set Command Group) (#173)

* Minor update + remove change log.

* Minor documentation update.

* Minor IT update.

* Rebase + Spotless.

* Minor IT test update.
@SanHalacogluImproving SanHalacogluImproving deleted the java/dev_SanH_add_Zmscore branch April 26, 2024 20:39
cyip10 pushed a commit that referenced this pull request Jun 24, 2024
* Java: Add Zmscore command. (Sorted Set Command Group) (#173)

* Minor update + remove change log.

* Minor documentation update.

* Minor IT update.

* Rebase + Spotless.

* Minor IT test update.
acarbonetto pushed a commit that referenced this pull request Jul 11, 2024
…client to properly take received IPs when DNS is expanded #173

Signed-off-by: barshaul <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants