FoundatioFx / Foundatio.Redis

Foundatio Redis
Apache License 2.0
98 stars 31 forks source link

redis cluster support #77

Closed splitice closed 1 year ago

splitice commented 1 year ago

This version will actually work against redis cluster (and non redis cluster).

Validated against a keydb based cluster CI environment and a redis-server standalone.

Key changes include:

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

CLAassistant commented 1 year ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

ejsmith commented 1 year ago

@splitice Thank you! I seem to recall there were other issues trying to make use of Redis cluster. Did you test the library pretty extensively on Redis cluster? Also, I forget if we test against both cluster and single server in our tests, but it would be good to test against both if this is fixing cluster issues.

splitice commented 1 year ago

@ejsmith My testing is limited to RedisQueue, thats the only component we need at this time