In this commit, first tried enabling caching by default, if neverExpire is false.
Unlimited caching is not a good idea for any dynamic application, especially dynamic configuration. So next tried turning the default duration when enabled, 30s is a reasonable default value. It will neither cause memory pressure nor cause long-term reading of expired configuration.
See gh-34172
Comment From: philwebb
I'd like to get a bit more information in #34172 before we consider changing any defaults.