using @ Transaction in @ Observes print warning log: WARN [io.agr.pool] (trans-consumer-4) Datasource '<default>': Closing open connection prior to commit

[Clever-Wang]

Describe the bug

Quarkus version ： 3.5.0

Expected behavior

No response

Actual behavior

No response

How to Reproduce?

No response

Output of uname -a or ver

No response

Output of java -version

No response

Quarkus version or git rev

No response

Build tool (ie. output of mvnw --version or gradlew --version)

No response

Additional information

No response

[yrodiere]

Thank you but I think I'll need a little bit more information, like the extensions you're using.

Please provide a simple project (as a zip, github repo, ...) that reproduces the problem?

[mmusgrov]

This problem looks like a duplicate of #18737

I have a fix for it (#34160 ) but it fails one of the panache tests and I haven't had a chance to fully investigate why the test fails ... but let me take another look at the failing test and I will update this thread when I've figured it out.

The java stacktrace for the successful case is https://pastebin.com/Lk9UpUGv and for the unsuccessful case it is https://pastebin.com/rVUUSFAy

[mmusgrov]

I found the issue with the PR, I wasn't re-throwing the exception during the before synchronization handler. I've updated my PR on the other issue with the fix so let's see how it does this time ...

[mmusgrov]

Actually although throwing the exception fixes the failing test it will stop any subsequent beforeCompletion synchronizations from running so we will need a different strategy, or maybe the test is wrong - I will investigate further.

[yrodiere]

This problem looks like a duplicate of #18737

Alright, closing as duplicate of #18737 then.

@Clever-Wang feel free to chime in if you think this is a different problem, otherwise let's continue the conversation on #18737.