HV-2116 Do not recreate PersistenceUtil in the built-in JPATraversableResolver

[marko-bekhta]

https://hibernate.atlassian.net/browse/HV-2116

[Please describe here what your change is about]

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on licensing, please check here.

---

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[gsmet]

Did you discuss this with the Hibernate ORM team? Because I think it's a bit weird to create a new instance each time in Persistence?

And the fact that we missed this is a good example of why this can be misleading.

[gsmet]

Note that it looks like something that should be merged and backported. I was just wondering if it could impact other projects out there.

[marko-bekhta]

No, I was only looking into that other Quarkus issue related to the resolver and noticed this, so I pushed the change to be sure I wouldn't forget about it. Thanks for the suggestion, I'll start a topic on Zulip once I finish with the other HV "backport" 😃 🙈

[gsmet]

Yeah, let's get it in though and backported because it allocates useless things in a relatively hot path.

[gsmet]

Good catch :)