-
Notifications
You must be signed in to change notification settings - Fork 37.7k
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
Support multiple TaskDecorators in TaskExecutionAutoConfiguration #23686
Labels
in: core
Issues in core modules (aop, beans, core, context, expression)
status: superseded
An issue that has been superseded by another
type: enhancement
A general enhancement
Comments
I've moved that suggestion to Spring Framework. If it turns out this gets added in the core container, we can make the necessary changes in Spring Boot. |
spring-projects-issues
added
the
status: waiting-for-triage
An issue we've not yet triaged or decided on
label
Sep 24, 2019
sbrannen
added
in: core
Issues in core modules (aop, beans, core, context, expression)
type: enhancement
A general enhancement
labels
Sep 24, 2019
ttddyy
added a commit
to ttddyy/spring-framework
that referenced
this issue
Sep 24, 2019
This is an old issue, any movement? I'm glad I googled, because I was about to run into the exact issue solved by this PR! |
sbrannen
changed the title
Multiple TaskDecorator support in TaskExecutionAutoConfiguration
Support multiple TaskDecorators in TaskExecutionAutoConfiguration
Jun 17, 2022
We'll discuss it within the team. |
Great! |
Closing in favor of PR #23692 |
snicoll
added
status: superseded
An issue that has been superseded by another
and removed
status: waiting-for-triage
An issue we've not yet triaged or decided on
labels
Aug 23, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
in: core
Issues in core modules (aop, beans, core, context, expression)
status: superseded
An issue that has been superseded by another
type: enhancement
A general enhancement
TaskExecutionAutoConfiguration
currently only take singleTaskDecorator
bean.This is because there is no implementation that represents multiple
TaskDeorator
s. (e.g.:CompositeTaskDecorator
).I handle this by creating
CompositeTaskDecorator
:I believe it is pretty common to have usecases to apply multiple TaskDecorators; so, it is nice if
CompositeTaskDecorator
is available in either spring or spring-boot.Then, if such class is available,
TaskExecutionAutoConfiguration
(TaskExecutorBuilder
) can detect multiple task decorators.The text was updated successfully, but these errors were encountered: