search

redis / go-redis

Redis Go client
https://redis.uptrace.dev
BSD 2-Clause "Simplified" License
19.81k stars 2.34k forks source link

Add RE tests #2847

Closed ofekshenawa closed 8 months ago

sjpotter commented 8 months ago

dont understand why so many of the tests (all?) in commands_test.go are "non redis enterprise". many of them seem like normal commands that should work fine.

ofekshenawa commented 8 months ago

@sjpotter, I have labeled tests as 'non-Redis Enterprise' when they involve commands unsupported in Redis Enterprise, such as ACL commands, config, etc., and tests that result in cross-slot violations

sjpotter commented 8 months ago

I'm surprised that commands like blpop/brpop aren't supported in redis enterprise? but perhaps I should have?

chayim commented 8 months ago

IMHO we need this https://github.com/RedisLabs/redis-ee-docker/pull/7

chayim commented 8 months ago

Willing to pull in for ee-docker#7 as a separate item.