You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The Pet Clinic depends on EhCache, but it doesn't even use it unless the "production" profile is enabled. And yet it costs a couple hundred ms apparently (see benchmarks at https://github.com/dsyer/spring-boot-startup-bench). This might be just a special case of too much condition processing on startup, but I did notice that all cache types are configured in CacheConfigurationImportSelector, even if they are switched off by condition processing, and maybe this is unnecessary.
The text was updated successfully, but these errors were encountered:
I noticed that too and we should revisit the condition to back-off sooner indeed. If you have an app with nothing particular on the classpath, the cache conditions always shows up which is not nice at all.
@dsyer I know you've been looking at pet clinic again recently. Did this issue end up being a quirk of the report (#13509) or was it legit? What are the steps we need to follow to replicate?
The Pet Clinic depends on EhCache, but it doesn't even use it unless the "production" profile is enabled. And yet it costs a couple hundred ms apparently (see benchmarks at https://github.com/dsyer/spring-boot-startup-bench). This might be just a special case of too much condition processing on startup, but I did notice that all cache types are configured in
CacheConfigurationImportSelector
, even if they are switched off by condition processing, and maybe this is unnecessary.The text was updated successfully, but these errors were encountered: