Skip to content

[WFLY-21044]: Not recovered transactions after crash of server with MDB.#19304

Merged
bstansberry merged 2 commits intowildfly:38.xfrom
ehsavoie:WFLY-21044_38x
Nov 2, 2025
Merged

[WFLY-21044]: Not recovered transactions after crash of server with MDB.#19304
bstansberry merged 2 commits intowildfly:38.xfrom
ehsavoie:WFLY-21044_38x

Conversation

@ehsavoie
Copy link
Member

  • Fixing how WildFly provides the recovery manager to Artemis
  • Removing the old MSC API usage to get the RecoveryService

Jira: https://issues.redhat.com/browse/WFLY-21044
Upstream: #19303

…MDB.

 * Fixing how WildFly provides the recovery manager to Artemis
 * Removing the old MSC API usage to get the RecoveryService

Jira: https://issues.redhat.com/browse/WFLY-21044

Signed-off-by: Emmanuel Hugonnet <ehugonne@redhat.com>
@jmesnil jmesnil added the 38.x label Oct 23, 2025
@rhusar rhusar changed the title [WFLY-21044-]: Not recovered transactions after crash of server with MDB. [WFLY-21044]: Not recovered transactions after crash of server with MDB. Oct 23, 2025
@bstansberry bstansberry added the Blocker Needs to be merged before the next release from the branch it targets. label Oct 27, 2025
@jmesnil jmesnil self-assigned this Oct 31, 2025
@@ -38,9 +34,6 @@ private static XAResourceRecoveryRegistry getXAResourceRecoveryRegistry() {
// This parsing isn't 100% ideal as it's somewhat 'internal' knowledge of the relationship between
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you remove this comment to be in sync with the required changed for #19303?

Copy link
Contributor

@bstansberry bstansberry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed the cruft code comment.

@bstansberry bstansberry merged commit 656ce14 into wildfly:38.x Nov 2, 2025
11 of 12 checks passed
@bstansberry
Copy link
Contributor

Thanks @ehsavoie and @jmesnil

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

Labels

Blocker Needs to be merged before the next release from the branch it targets.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants