SetArgs GET with NX (Redis 7)

[terrettaz]

Feature Request

Command SET arguments change

Is your feature request related to a problem? Please describe

Indeed since version 7.0.0, redis added the possibility to use both options GET and NX together. We'd like to have this small feature to use it in our projects instead of internally extending the class SetArgs
https://redis.io/docs/latest/commands/set/#options

Describe the solution you'd like

Adding the possibility to specify the option GET with NX.

I started the work here main...terrettaz:lettuce:SetArgs-GET
If accepted this is ready to be requested

Describe alternatives you've considered

N/A

Teachability, Documentation, Adoption, Migration Strategy

N/A

[tishun]

Hey @terrettaz ,

Thanks for the suggestion, please file a PR and we will go over it.

From what I see seems like a straightforward improvement.

[terrettaz]

Hey @tishun Thanks for quick reply.

You will see the PR here #3411

I based on main branch but you may want to cherry-pick also for other branch like 6.8

[tishun]

Hey @terrettaz,

Is this solution different in any way than the solution in #1442 ?

[terrettaz]

Hi,
I've been silent on this topic since long time, sorry about this (was assigned to different stuff)
Yes I think this #1442 should be the solution, I didn't see this method before.
I will need to test in our application soon.

[tishun]

Thanks for confirming!

[tishun]

Please reopen in case there is still something missing.