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

[DRAFT] WIP on Concurrency work #1090

Closed
wants to merge 14 commits into from

Conversation

jgallimore
Copy link
Contributor

Rebased some old code I had prior to CDI 4 to implement the concurrency spec changes. Still working on unit test failures, and likely isn't complete.

I'll keep plugging away, but equally happy for someone else to incorporate this into another PR, or trash this and do something else, or whatever.

@rzo1
Copy link
Contributor

rzo1 commented Jan 8, 2024

Fyi @tandraschko

@rzo1
Copy link
Contributor

rzo1 commented Jan 16, 2024

Let's see how it looks for a full build, so people can get a chance to dig into test failures: https://ci-builds.apache.org/job/Tomee/job/pull-request-manual/64/

@tandraschko
Copy link
Member

which failing tests are related to this change?
only org.apache.openejb.arquillian.ManagedExecutorServiceGetPrincipalInTaskTest ?

@tandraschko
Copy link
Member

tandraschko commented Jan 17, 2024

i dont think that JsfTest stuff is related to this change? and they run on local?

@rzo1
Copy link
Contributor

rzo1 commented Jan 17, 2024

Ok, didn't test them locally, but was of the impression, that the JSF stuff is related as @jgallimore changed the context service stuff inside this PR ;-)

rzo1 added 2 commits January 18, 2024 10:10
# Conflicts:
#	container/openejb-core/src/main/java/org/apache/openejb/config/AnnotationDeployer.java
@rzo1
Copy link
Contributor

rzo1 commented Jan 18, 2024

The content is here: #1103

@rzo1 rzo1 closed this Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants