Support IFEQ option for SET command

[hpatro]

Describe the feature

Support IFEQ option for SET command with valkey-glide clients. Additional option is planned to be added to SET command in Valkey 8.1

valkey-io/valkey-doc#189

Use Case

Explained in valkey-io/valkey#1215 and implemented in valkey-io/valkey#1324

Checklist

• Node SET IFEQ command implemented in node + tests #2909
• python
• Java SET IFEQ command implemented in Java + tests  #2978
• Go Go: Add IFEQ to SetOptions #3098

[avifenesh]

Hi @hpatro ☺️
Thanks for the heads up! Will add it to our planning.
If you want to contribute it, i will be happy to give you a tour.

[hpatro]

Hi @hpatro ☺️ Thanks for the heads up! Will add it to our planning. If you want to contribute it, i will be happy to give you a tour.

Thanks @avifenesh. I also wanted to understand the architecture of valkey glide better. Will reachout.

[avifenesh]

Hi @hpatro ☺️ Thanks for the heads up! Will add it to our planning. If you want to contribute it, i will be happy to give you a tour.

Thanks @avifenesh. I also wanted to understand the architecture of valkey glide better. Will reachout.

You can find me both on Valkey slack as well in AWS.
Feel free when you feel like, always happy to chat :)

[Maayanshani25]

The implementation requires modifying the set command and adding separate and integrated test cases with other SET options like GET and EX.
Estimation: 6 days.

[avifenesh]

@Maayanshani25 Hi brother, hope you're enjoying your trip! :)
Please note that I'm assigning the Python implementation to @Angraybill.
If you want to talk about it, ping ♥️

[avifenesh]

@hpatro Until now, the options for conditional change values were the same for SET as for GEOADD and ZADD.
IFEQ is for SET only, require some additions in the type system, since we need to enforce using IFEQ in SET only and not in the other. Hence, it cannot be part of the ConditionalChange Enum.
Before we create the new changes, for planing it right for the future to avoid breaking changes, or extra and redundant types - Is it possible that it will be added to the other commands?
I understand that those will require deep equal check and not just equal, and you can't see the future, but maybe you have some clue about the possibilities.

[tjzhang-BQ]

Merged golang changes to the main branch, closing this issue now