jdaugherty opened a new pull request, #16324:
URL: https://github.com/apache/grails-core/pull/16324
Follow-up to #16299 addressing review feedback.
Under a DevTools restart the thread context class loader is not always the
RestartClassLoader. Tomcat swaps
in TomcatEmbeddedWebappClassLoader during context start, which delegates to
the RestartClassLoader as its
parent. A check that matched only the loader's own type fell back to the
base loader in that window, which
is how the Hibernate 5 SessionFactory on 8.0.0-M5 ended up with base-loader
entity classes (#16287).
Current 8.0.x only avoids it because the datastore is now constructed before
Tomcat starts.
- Walk the loader parent chain in DevToolsClassLoaders; keep a fallback that
already sees the restart loader.
- Hibernate 5 mirrors Hibernate 7: connection source is the DataSource of
record, the context supplies the
class loader. Removes the duplicated bean-name derivation.
- Drop the unreleased resolve() alias and the guide paragraph.
- Specs cover the container-loader condition, which the previous specs did
not exercise.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
