Skip to content

WFLY-21293 Session.requestDone(...) invoked outside of ControlPoint boundary#19491

Merged
bstansberry merged 2 commits intowildfly:mainfrom
pferraro:WFLY-21293
Jan 14, 2026
Merged

WFLY-21293 Session.requestDone(...) invoked outside of ControlPoint boundary#19491
bstansberry merged 2 commits intowildfly:mainfrom
pferraro:WFLY-21293

Conversation

@pferraro
Copy link
Contributor

https://issues.redhat.com/browse/WFLY-21293

Should resolve the vast majority of intermittent distributable web test failures in clustering testsuite.

@pferraro
Copy link
Contributor Author

pferraro commented Dec 18, 2025

Let me convert to a draft while I sort out the corner cases that are clearly not being handled.

@pferraro pferraro marked this pull request as draft December 18, 2025 00:19
@pferraro pferraro marked this pull request as ready for review December 18, 2025 00:46
@pferraro pferraro force-pushed the WFLY-21293 branch 2 times, most recently from 3ec7ab5 to b2e837b Compare December 18, 2025 08:21
@pferraro pferraro marked this pull request as draft December 18, 2025 12:14
@pferraro pferraro marked this pull request as ready for review December 18, 2025 14:17
@pferraro pferraro marked this pull request as draft December 18, 2025 14:19
@pferraro
Copy link
Contributor Author

N.B. I don't like this idea of deferring ControlPoint.requestDone() to the Session.requestDone(...).
I'm exploring an alternate solution that effectively moves Session.requestDone(...) handling to a ExchangeCompletionListener.

@pferraro pferraro marked this pull request as ready for review December 18, 2025 18:39
@pferraro
Copy link
Contributor Author

OK - this should be ready for review now.

@rhusar rhusar requested a review from bstansberry December 23, 2025 08:41
@bstansberry bstansberry added the 39.x WildFly 39 label Jan 7, 2026
This was referenced Jan 13, 2026
@bstansberry bstansberry merged commit b2b54e3 into wildfly:main Jan 14, 2026
13 of 14 checks passed
@bstansberry
Copy link
Contributor

Thanks @pferraro @rhusar and @fl4via

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

39.x WildFly 39

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants