Skip to content

Conversation

@sbera13
Copy link

@sbera13 sbera13 commented Dec 3, 2025

This PR resolves issue #48033. It has the following changes

  • Introduced new property propagateContext
  • Added bean for ContextPropagatingTaskDecorator based on the property
  • Modified unit tests

Copy link
Member

@snicoll snicoll left a 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.

@snicoll snicoll added the status: waiting-for-feedback We need additional information before we can continue label Dec 3, 2025
@sbera13 sbera13 requested a review from snicoll December 4, 2025 13:58
@philwebb philwebb changed the title Property to register ContextPropagatingTaskDecorator bean #48033 Add property to automatically register a ContextPropagatingTaskDecorator bean Dec 4, 2025
@snicoll
Copy link
Member

snicoll commented Dec 8, 2025

@sbera13 the build is failing, can you please check?

@snicoll
Copy link
Member

snicoll commented Dec 18, 2025

Alright I am taking over.

@snicoll snicoll self-assigned this Dec 18, 2025
@snicoll snicoll added type: enhancement A general enhancement and removed status: waiting-for-feedback We need additional information before we can continue status: waiting-for-triage An issue we've not yet triaged labels Dec 18, 2025
@snicoll snicoll added this to the 4.1.x milestone Dec 18, 2025
@snicoll
Copy link
Member

snicoll commented Dec 19, 2025

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 snicoll closed this Dec 19, 2025
@snicoll snicoll removed this from the 4.1.x milestone Dec 19, 2025
@snicoll snicoll added the status: declined A suggestion or change that we don't feel we should currently apply label Dec 19, 2025
@sbera13
Copy link
Author

sbera13 commented Dec 19, 2025

@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.

snicoll added a commit to snicoll/spring-boot that referenced this pull request Dec 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: declined A suggestion or change that we don't feel we should currently apply type: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants