<div dir="ltr">The 
JCAOrderedLastSynchronizationL<wbr>ist is intended to guarnatee the ordering in the JVM so it seems very likely it should be used. I wonder why the wrapper is not being injected into JPA and JCA?<br></div><div class="gmail_extra"><br><div class="gmail_quote">On 14 November 2018 at 17:34, Scott Marlow <span dir="ltr">&lt;<a href="mailto:smarlow@redhat.com" target="_blank">smarlow@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I made a local change to JCAOrderedLastSynchronizationL<wbr>ist, to have the <br>
JPA container (TransactionUtil$<wbr>SessionSynchronization) afterCompletion() <br>
always be run after Hibernate afterCompletion() but before the JCA <br>
afterCompletion().  As I am implementing EntityManager caching [1] and <br>
want to ensure that the EntityManager instances are only returned to the <br>
cache after the EntityManager is expected to be available for reuse.<br>
<br>
In my debugging, I noticed that the JCAOrderedLastSynchronizationL<wbr>ist <br>
only has the Hibernate synchronizations being added but not the JCA or <br>
JPA container synchronizations.  [2] shows that the JCA + JPA <br>
Synchronizations are registered via <br>
org.wildfly.transaction.<wbr>client.<wbr>ContextTransactionSynchronizat<wbr>ionRegistry <br>
and Hibernate ORM Synchronization is registered via the <br>
org.jboss.as.txn.service.<wbr>internal.tsr.<wbr>TransactionSynchronizationRegi<wbr>stryWrapper<br>
<br>
Should <br>
org.wildfly.transaction.<wbr>client.<wbr>ContextTransactionSynchronizat<wbr>ionRegistry <br>
be changed to use JCAOrderedLastSynchronizationL<wbr>ist as well, so that <br>
IronJacamar (JCA) Synchronizations are run in the correct order?<br>
<br>
Should we change the JPA subsystem/container to also ensure that <br>
Hibernate ORM 5.3 uses the ContextTransactionSynchronizat<wbr>ionRegistry <br>
instead of the <br>
org.jboss.as.txn.service.<wbr>internal.tsr.<wbr>TransactionSynchronizationRegi<wbr>stryWrapper? <br>
<br>
<br>
Is there a WildFly service that represents the <br>
org.wildfly.transaction.<wbr>client.<wbr>ContextTransactionSynchronizat<wbr>ionRegistry <br>
lifecycle?  The JPA persistence unit service (or global JPA service) <br>
should depend on that service, so that applications undeploy if the TSR <br>
is stopped.<br>
<br>
[3] is also related to the the [1] effort and would be impacted by the <br>
above mentioned changes.<br>
<br>
Scott<br>
<br>
[1] EntityManager caching <a href="https://issues.jboss.org/browse/WFLY-11233" rel="noreferrer" target="_blank">https://issues.jboss.org/<wbr>browse/WFLY-11233</a><br>
<br>
[2] <a href="https://paste.fedoraproject.org/paste/-wHaYB3mzR2yTXTfU3LbZA" rel="noreferrer" target="_blank">https://paste.fedoraproject.<wbr>org/paste/-<wbr>wHaYB3mzR2yTXTfU3LbZA</a><br>
<br>
[3] Ensure that Hibernate uses direct reference to TSR for better <br>
performance <a href="https://issues.jboss.org/browse/WFLY-11243" rel="noreferrer" target="_blank">https://issues.jboss.org/<wbr>browse/WFLY-11243</a><br>
______________________________<wbr>_________________<br>
wildfly-dev mailing list<br>
<a href="mailto:wildfly-dev@lists.jboss.org">wildfly-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/wildfly-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/<wbr>mailman/listinfo/wildfly-dev</a><br>
</blockquote></div><br></div>