Skip to content
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

Improvement/xml resource loader startup #301

Merged

Conversation

groldan
Copy link
Member

@groldan groldan commented Feb 8, 2023

Improve startup time by caching XML classpath resources

A ~1.5 seconds startup time improvement, provided by FilteringXmlBeanDefinitionReader caching parsed XML
documents by Resource URI, since many configurations can try to load different sets of beans from the same
xml document, and all the classpath Resource's to avoid loading them all for each location.

FilteringXmlBeanDefinitionReaderAutoConfiguration clears the caches once the application context is refreshed.

A ~1.5 seconds startup time improvement, provided by
`FilteringXmlBeanDefinitionReader` caching parsed XML
documents by Resource URI, since many configurations
can try to load different sets of beans from the same
xml document, and all the classpath Resource's to
avoid loading them all for each location.

`FilteringXmlBeanDefinitionReaderAutoConfiguration` clears
the caches once the application context is refreshed.
@groldan groldan added the enhancement New feature or request label Feb 8, 2023
@groldan groldan self-assigned this Feb 8, 2023
@groldan groldan merged commit b707879 into geoserver:main Feb 8, 2023
@groldan groldan deleted the improvement/xml_resource_loader_startup branch February 8, 2023 01:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant