Skip to content

Update `RedisClient.expire` to no longer use `deadline` terminology.

Nathan Harris requested to merge expire-timeout into 47-proposal-feedback

Motivation:

During proposal review, it was appropriately pointed out that RedisClient.expire incorrectly mixes 'deadline' and 'timeout' terminology.

Modifications:

  • Change references of 'deadline' to 'timeout' to follow Redis' established semantics for 'EXPIRE'
  • Add additional unit test for RedisClient.expire

Result:

RedisClient.expire should now be more clear as to its semantics and not mix terminology incorrectly.

This contributes to issue #47 (closed)

Merge request reports