-
Notifications
You must be signed in to change notification settings - Fork 41.7k
Add property to automatically register a ContextPropagatingTaskDecorator bean #48385
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
snicoll
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR. Please run the build locally to make sure formatting and checkstyle checks are passing.
I've added a few comments for your consideration.
...igure/src/main/java/org/springframework/boot/autoconfigure/task/TaskExecutionProperties.java
Outdated
Show resolved
Hide resolved
...re/src/main/java/org/springframework/boot/autoconfigure/task/TaskExecutorConfigurations.java
Outdated
Show resolved
Hide resolved
...re/src/main/java/org/springframework/boot/autoconfigure/task/TaskExecutorConfigurations.java
Outdated
Show resolved
Hide resolved
...re/src/main/java/org/springframework/boot/autoconfigure/task/TaskExecutorConfigurations.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Sandipan <bsandipan99@gmail.com>
Signed-off-by: Sandipan <bsandipan99@gmail.com>
Signed-off-by: Sandipan <bsandipan99@gmail.com>
|
@sbera13 the build is failing, can you please check? |
|
Alright I am taking over. |
|
Sorry @sbera13 but reviewing the PR there's not much that's left from the original contribution. The code doesn't compile, checkstyle is failing and the tests are all failing as well. Going forward, please run the build on the command line and make sure the code you've added doesn't break it. |
|
@snicoll My apologies for not being active enough on this. Thanks for taking care of this. Will keep your suggestions in mind from next time. |
This PR resolves issue #48033. It has the following changes
propagateContextContextPropagatingTaskDecoratorbased on the property