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

[32.x][WFLY-12329] managed executor service's execute(), submit() and schedule*() should use RequestController#forceBeginRequest() only when starting #17882

Closed
wants to merge 1 commit into from

Conversation

emmartins
Copy link
Contributor

@emmartins emmartins commented May 10, 2024

Issue: https://issues.redhat.com/browse/WFLY-12329
Main branch PR: #17881


More information about the wildfly-bot[bot]

@wildfly-bot wildfly-bot bot requested a review from bstansberry May 10, 2024 11:34
@bstansberry bstansberry added the 32.x WildFly 32 label May 14, 2024
@bstansberry
Copy link
Contributor

/retest

@bstansberry
Copy link
Contributor

I don't think this is a good choice for a micro. Even if there were no concerns about the solution (I have some; see the other PR), I think a behavior change like this should bake a while in main.

@asoldano asoldano added the hold PR should not be merged for some reason. label May 17, 2024
@emmartins emmartins changed the title [32.x][WFLY-12329] Align ManagedScheduledExecutorService schedule...() beha… [32.x][WFLY-12329] managed executor service's execute(), submit() and schedule*() should use RequestController#forceBeginRequest() only when starting May 22, 2024
…ule*() should use RequestController#forceBeginRequest() only when starting
@bstansberry bstansberry requested review from bstansberry and removed request for bstansberry May 30, 2024 16:31
@emmartins
Copy link
Contributor Author

closing this one since 32.0.1 is released

@emmartins emmartins closed this Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
32.x WildFly 32 hold PR should not be merged for some reason.
Projects
None yet
3 participants