Skip to content

[WFLY-21409] Update org.jboss.as.jpa.container.NonTxEmCloser.popCall to check for null container managed stateless session/entity manager#19584

Merged
rhusar merged 1 commit intowildfly:39.xfrom
scottmarlow:WFLY-21409_39.x_check_for_null_NonTxEmCloser_popCall
Feb 1, 2026
Merged

[WFLY-21409] Update org.jboss.as.jpa.container.NonTxEmCloser.popCall to check for null container managed stateless session/entity manager#19584
rhusar merged 1 commit intowildfly:39.xfrom
scottmarlow:WFLY-21409_39.x_check_for_null_NonTxEmCloser_popCall

Conversation

@scottmarlow
Copy link
Contributor

https://issues.redhat.com/browse/WFLY-21409
backport of #19578

This is to address a minor (e.g. ignored) NPE that only occurs with TRACE logging enabled for Persistence:

2026-01-29 11:02:32,150 TRACE [org.jboss.as.jpa] (default task-2) Could not close (non-transactional) container managed stateless session. This shouldn't impact application functionality (only read operations occur in non-transactional mode): java.lang.NullPointerException: Cannot invoke "java.lang.AutoCloseable.close()" because "statelessSession" is null
at [org.jboss.as.jpa@40.0.0.Beta1-SNAPSHOT](mailto:org.jboss.as.jpa@40.0.0.Beta1-SNAPSHOT)//org.jboss.as.jpa.container.NonTxEmCloser.popCall(NonTxEmCloser.java:61)

…to check for null container managed stateless session/entity manager

Signed-off-by: Scott Marlow <scott.marlow@ibm.com>
@rhusar rhusar self-assigned this Jan 30, 2026
@rhusar rhusar added the 39.x WildFly 39 label Jan 30, 2026
Copy link
Member

@rhusar rhusar left a comment

Choose a reason for hiding this comment

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

LGTM - backport of a null safety fix; approved also by Brian upstream.

@rhusar rhusar merged commit de4e1b6 into wildfly:39.x Feb 1, 2026
12 checks passed
@rhusar
Copy link
Member

rhusar commented Feb 1, 2026

Thanks @scottmarlow !

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.

2 participants