Skip to content

Convert date to string as args for Redis commands

Sylvester Chin requested to merge sc1-date-tostring-redis into master

What does this MR do and why?

In redis gem v5, argument types are mainly restricted to Numbers and String. This MR performs argument type conversion instead of letting the redis gem handle it (in redis v4.8.0, Redis::Connection::CommandHelper handles it).

This reduces the MR-diff for the gem upgrade MR (draft in !134862 (diffs)).

Issue: gitlab-com/gl-infra/scalability#2586 (closed)

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports