-
-
Notifications
You must be signed in to change notification settings - Fork 454
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider updating to StackExchange.Redis 2.1 #309
Comments
Is there any breaking change in the 2.1 client which prevents you from using that version together with CacheManager? |
Do you mean referencing the StackExchange.Redis Nuget package directly, essentially overriding the reference in CacheManager? |
Yeah exactly. That's usually how you do that with Nugets, same as with .NET frameworks, I usually target slightly lower versions in libraries so that I don't force consumers to upgrade. And, over time, dependencies might get new versions and I don't want/have to release a new version of a library just because one dependency got a new minor version. Just give it a try. |
Can confirm that we're running CacheManager 1.2 with SE.Redis 2.1 in production and haven't seen any issues yet. |
@keithl8041 Sounds similar to our issues, yes. Although we see it across a bunch of different application services (web APIs) and I'm not sure of the deployment structure, whether the affected APIs at any time are in the same app service instance or not. I'll ask our devops folks about that the next time it's reported/observed. |
We are encountering an existing a bug in StackExchange.Redis on production We use the version 2.0.0-beta-1629 with StackExchange.Redis 2.0.513 If you can create a new release based on StackExchange.Redis 2.1 version, it would work for us. |
Updated the version of StackExchange.Redis in #337 |
StackExchange.Redis is now at version 2.1.x, and there are useful fixes in there. Most notably to me is fixing some intermittent connectivity problems with Azure Redis. That is an issue that my team has been seeing over the past year or so and caused me some sleepless nights (I work on a very large platform for a very large enterprise client, heavily invested in Azure).
Will you consider releasing an updated version of CacheManager.StackExchange.Redis? If I was willing to submit a PR would that change your answer?
The text was updated successfully, but these errors were encountered: