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

precontextualized tasks #6396

Merged
merged 3 commits into from
Feb 6, 2019

Conversation

njr-11
Copy link
Contributor

@njr-11 njr-11 commented Feb 4, 2019

When ThreadContext is used to precontextualized Runnable and Callables that are supplied to ManagedExecutor.submit/execute/invoke methods, or to ManagedScheduledExecutorService.schedule methods, ensure that the already-captured thread context is used rather than capturing context again per the managed executor's configuration.

@LibbyBot
Copy link

LibbyBot commented Feb 4, 2019

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 14 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

  • 2 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

@njr-11
Copy link
Contributor Author

njr-11 commented Feb 4, 2019

#build

@LibbyBot
Copy link

LibbyBot commented Feb 4, 2019

Please code review feature-related files, @OpenLiberty/delivery-approvers

@LibbyBot
Copy link

LibbyBot commented Feb 4, 2019

Your personal build request is at https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_uGq5ECi-EemoQsY-pF-VMw

Target locations of links might be accessible only to IBM employees.

@njr-11
Copy link
Contributor Author

njr-11 commented Feb 6, 2019

build has 14 test failures, which are duplicates of preexisting issues in unrelated test buckets

@njr-11 njr-11 merged commit 926ec3d into OpenLiberty:integration Feb 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants