redis / riot

🧨 Get data in & out of Redis with RIOT
http://redis.github.io/riot
Apache License 2.0
260 stars 38 forks source link

Bug: Validation check for `set` types fails unexpectedly #119

Closed samongyr-sq closed 1 year ago

samongyr-sq commented 1 year ago

Validation fails unexpectedly on sets with >1 member loaded via the replicate command with --mode snapshot

$ riot -d <target> replicate <target> --mode compare --show-diffs 2>&1 | grep WARNING
20:12:40.923 WARNING com.redis.riot.cli.Replicate   : Value mismatch for string my_stringXYZ at offset 17
20:12:40.923 WARNING com.redis.riot.cli.Replicate   : Value mismatch for set my_set: [] <> []
20:12:41.246 WARNING com.redis.riot.cli.common.KeyComparisonStepListener    : Verification failed: OK=2 Missing=0 Values=2 TTLs=0 Types=0

Checking the mismatch via redis-cli proves that the sets should be equal

$ redis-cli <source> smembers my_set
1) "000000000021"
2) "000000000026"

$ redis-cli <target> smembers my_set
1) "000000000026"
2) "000000000021"

I would expect the validation check for my_set to pass

jruaux commented 1 year ago

🎉 This issue has been resolved in v3.1.1 (Release Notes)