Hello @soloestoy, thanks for this PR! I think that's going to be more complex than that :-(
Usually we rewrite expires as EXPIREAT in the AOF channel, and relative EXPIRE in the replication channel. So this patch would not be enough. I think we may need something else here, probably a new SET option that does not touch the expire would be ideal in such cases. What do you think?

a new SET option that does not touch the expire would be ideal in such cases

I prefer this, and some users wait this option for a long time.

Awesome @soloestoy, what about KEEPTTL?

Emm... seems too long? Now we have 4 options NX|XX|EX|PX, to keep the format how about UX means Untouch eXpire

@soloestoy I don't think it's a good idea to continue with the short ones, I used NX because we historically had the "SETNX" command, then XX. EX / PX in retrospect are errors of mine, they are not understandable. UX would be IMHO the next error, so I would go for KEEPTTL.

/me just realized there will never be THX flag, shame

@soloestoy are you willing to update the PR with the proposed solution, or I may do it. Asking because RC1 is very near! :-D Thanks.

Hi @antirez I open a new PR #6679 , please check ^_^

Thanks @soloestoy! Merged :-) Closing this.