Description
Steffen Ryll opened SPR-13610 and commented
We have a pretty large ApplicationContext with about 7600 Beans and startup times have become an issue, now at 5.5 min just for the ApplicationContext. We observed that all the steps in AbstractApplicationContext#refresh before #preInstantiateSingletons() is called take about 90 seconds (excluding the time it takes to load all bean definition XML files). About 81 seconds of that time seems to be spent in the method DefaultListableBeanFactory#getBeanNamesForType.
In order to better understand where time is spent, we've build a wrapper around DefaultListableBeanFactory that measures the execution time of calls to #createBean, #getBeanNamesForType and #applyBeanPostProcessors*Initialization/Instantiation. This is the tooling we used to get the numbers mentioned above.
Closer inspection shows that runtime performance of #getBeanNamesForType depends heavily on the flag #isConfigurationFrozen - as long as it is set to false during AppCtx refresh, each call takes about 10 seconds, when it is set to true, calls typically take only ~100 ms.
A particular issue seems to me, that #getBeanNamesForType is called at least three times for the type BeanDefinitionRegistryPostProcessor, so that's more than 30 seconds in our scenario. I found that this has been implemented as part of #15258, and comments in the code of PostProcessorRegistrationDelegate indicate that calling it three times is "by design". But that seems like a waste of time to me, given that these calls take rather long.
Please let's discuss which optimizations would be feasible here within the boundaries of the involved interface contracts.
Affects: 4.0.7, 4.2.1
Issue Links:
- Spring-specific index file for component candidate classes [SPR-11890] #16509 Spring-specific index file for component candidate classes
- Allow BeanDefinitionRegistryPostProcessor to register other BeanDefinitionRegistryPostProcessors [SPR-10630] #15258 Allow BeanDefinitionRegistryPostProcessor to register other BeanDefinitionRegistryPostProcessors
- Background initialization option for JPA EntityManagerFactory / Hibernate SessionFactory [SPR-13732] #18305 Background initialization option for JPA EntityManagerFactory / Hibernate SessionFactory
- Autowiring with @Autowired/@Inject get much slower than with @Resource as the number of classes increases [SPR-13086] #17677 Autowiring with
@Autowired/@Injectget much slower than with@Resourceas the number of classes increases - Efficient type matching support for large number of beans [SPR-17528] #22060 Efficient type matching support for large number of beans
4 votes, 10 watchers