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

Should failures in the lazy loading process of beans after refresh also be recorded in errors? #1400

Closed
qnnn opened this issue Sep 27, 2024 · 1 comment

Comments

@qnnn
Copy link
Contributor

qnnn commented Sep 27, 2024

Is your feature request related to a problem? Please describe.

When GenericScope#destroy is executed, it checks the execution result of the bean's destroy callback and records exceptions in the errors map. However, during lazy loading of the bean, the result of bean loading is not checked. Shouldn't we also record bean loading failures in the errors map?

Additionally, can we add an eager switch for refreshing, similar to ConfigurationProperties, to immediately create the bean after destruction? This way, we can determine the refresh result of the bean earlier.

@qnnn
Copy link
Contributor Author

qnnn commented Sep 27, 2024

The implementation for recording bean loading failures in errors already exists.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants