Unverified Commit 690ef363 authored by debing.sun's avatar debing.sun Committed by GitHub
Browse files

Fix HFE notification test timing caused by field expiration in hexpire command (#13325)

In the old test, we give the `hexpire` a very short expire time, which
caused the filed to be deleted by the time `hpersist` command was
executed. As a result, the `hpersist` command won't be able to give a
`hpersist` notification, leading to test stuck.

fail CI:
https://github.com/redis/redis/actions/runs/9342175887/job/25709886471
parent 9a2c6ba4
...@@ -364,7 +364,7 @@ start_server {tags {"pubsub network"}} { ...@@ -364,7 +364,7 @@ start_server {tags {"pubsub network"}} {
r hincrby myhash yes 10 r hincrby myhash yes 10
r hexpire myhash 999999 FIELDS 1 yes r hexpire myhash 999999 FIELDS 1 yes
r hexpireat myhash [expr {[clock seconds] + 999999}] NX FIELDS 1 no r hexpireat myhash [expr {[clock seconds] + 999999}] NX FIELDS 1 no
r hpexpire myhash 5 FIELDS 1 yes r hpexpire myhash 999999 FIELDS 1 yes
r hpersist myhash FIELDS 1 yes r hpersist myhash FIELDS 1 yes
assert_encoding $type myhash assert_encoding $type myhash
assert_equal "pmessage * __keyspace@${db}__:myhash hset" [$rd1 read] assert_equal "pmessage * __keyspace@${db}__:myhash hset" [$rd1 read]
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment