Support different read mode in RedisHybridCacheClient

[zinniawang149]

At the moment, Even if we setup the Read mode in RedisHybridCacheClientOptions while creating the RedisHybridCacheClient instance, it doesn't work when we access the Redis.

The reason for that is because when we call the base object like this, we didn't pass the options.ReadMode in.

Foundatio.Redis/src/Foundatio.Redis/Cache/RedisHybridCacheClient.cs

Line 9 in ceb704b

: base(new RedisCacheClient(o => o

So the base readmode is always CommandFlags.None.

Foundatio.Redis/src/Foundatio.Redis/Cache/RedisCacheClientOptions.cs

Line 17 in ceb704b

public CommandFlags ReadMode { get; set; } = CommandFlags.None;

Here is the example before/after the code update when we set the read mode as PreferReplica, the GetTypeCmds indicates the total number of read-only type commands. This is derived from the Redis commandstats statistic by summing all of the read-only type commands (get, hget, scard, lrange, and so on.):

The 001 nodes of both shards are the masters:

Before the code update, we can see all the read call still goes to the master node.

After the code update, the application starts fetching data from read replica.

[CLAassistant]

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.

---

Zinnia Wang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[CLAassistant]

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.

---

Zinnia Wang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[zinniawang149]

Have signed the Contributor License Agreement.

[niemyjski]

Thanks for the PR! Also, those charts were awesome, would you mind sharing more on this (do you have a script that generates images automatically? I've been needing to look to see if there is a good way to do this with otel.):

the GetTypeCmds indicates the total number of read-only type commands. This is derived from the Redis commandstats statistic by summing all of the read-only type commands (get, hget, scard, lrange, and so on.):

[zinniawang149]

@niemyjski
Thanks for accepting the change. The charts were actually screenshots I took from AWS CloudWatch metrics for Elasticache haha.

Also, wondering if you can release a new version? Because I need this change to improve the performance of our system.

[niemyjski]

Please use a nightly feed (would be good to see if you see any other issues), I need to check to see what else we need to do before next release.

[zinniawang149]

@niemyjski I tried to run this cmd: dotnet add package Foundatio.Redis --version 10.7.1-alpha.0.1
https://github.com/FoundatioFx/Foundatio.Redis/pkgs/nuget/Foundatio.Redis/189858430
but got the error of Package 'Foundatio.Redis' is incompatible with 'all' frameworks in project
It looks to me like it is unable to find package Foundatio.Redis with version (>= 10.7.1-alpha.0.1) on Nuget

Can you see if it is working on your side?

[niemyjski]

Do you have a package source for the nighties defined? https://github.com/exceptionless/Exceptionless/blob/883d75b47a1bab7009e960a615429223162f5be4/NuGet.Config#L4

[zinniawang149]

Right. It's working! Thanks for your help.