jdaugherty commented on PR #16631: URL: https://github.com/apache/grails-core/pull/16631#issuecomment-6402662376
@jamesfredley one follow-up to the statement-inspector item, in ea3558f3c6 (still a single commit). An application inspector could also be lost when a `configClass` set it on the service registry builder, by overriding `createStandardServiceRegistryBuilder`. GORM applies the configuration's properties, which held the capture, over the builder's settings, so the capture replaced the inspector. This affected Hibernate 5 and 7. The capture now picks the inspector to wrap when the session factory is built, from the merged registry settings and, on Hibernate 7, the inspector set on the configuration. A new test runs on both versions for each way of setting an inspector, and checks that the inspector that runs with capture on is the one that runs with it off. Details are on the thread. -- 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]
