- Issue created by @pcambra
- Merge request !12Issue #3527091: Fail gracefully when Redis not available → (Merged) created by pcambra
- 🇩🇪Germany jurgenhaas Gottmadingen
The idea is great, but the implementation should be doing it such that we don't have duplicate lines of code just next to each other. The existing fallback should be used if the Redis approach fails.
Please update the MR so that it works against the 1.2.x branch. Development always works in the latest branch and if it's a bug fix, it will then be back ported to previous supported versions.
- 🇪🇸Spain pcambra Asturies
I'm not using D11 yet :( it'd be great if 1.2 would have support for D10
- 🇩🇪Germany jurgenhaas Gottmadingen
Doesn't matter. As I mentioned before, we can back port to older versions. But development always works on the latest branch.
- 🇩🇪Germany jurgenhaas Gottmadingen
It's better, just left 1 suggestion on how this could be easier to read.
-
jurgenhaas →
committed fd4bb391 on 1.2.x authored by
pcambra →
Issue #3527091: Fail gracefully when Redis not available
-
jurgenhaas →
committed fd4bb391 on 1.2.x authored by
pcambra →
-
jurgenhaas →
committed fb83e6b5 on 1.1.x
Issue #3527091: Fail gracefully when Redis not available (cherry...
-
jurgenhaas →
committed fb83e6b5 on 1.1.x
- 🇩🇪Germany jurgenhaas Gottmadingen
Merged and back ported. Thank you @pcambra for your contribution.