-
Notifications
You must be signed in to change notification settings - Fork 38.5k
Add property accessors to ExecutorConfigurationSupport
#34034
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
Conversation
Why is that? And why do you need the bean's name? |
ExecutorConfigurationSupport.
ExecutorConfigurationSupport.
ExecutorConfigurationSupport
Because I want to print executor related information on the actuator endpoint, I need to obtain beanName as the key. Of course, this is not necessary, and I can also obtain it through other means. |
@lucky8987 you need to reset your "dev" branch to what you intend to submit in this PR. Right now there are a number of unrelated commits in it. |
Sorry, my other change should not have been submitted here. I have now reset this branch and I really hope that this PR can be merged. |
Thanks for the suggestion but the flag is a configuration shortcut to enable VT. We can't ensure it has the parity that you expect (i.e. it's not reliable to rely on the internal boolean flag to determine that VT is enabled). |
Sometimes I need to check if the
ExecutorConfigurationSupport
executor has enabled virtual threads, so I added the corresponding getter methods.