rzo1 opened a new pull request, #212: URL: https://github.com/apache/openjpa/pull/212
Applies @rmannibucau suggestion from #210: the candidate loaders were collected into a `MultiClassLoader` and unwrapped again immediately, which is a surprising use of a class loader as a container. They are now collected into a `List<ClassLoader>` and the array is built from it. The container only existed to keep the protected hook that contributes the loaders, so that hook now takes a `List<ClassLoader>`. The `MultiClassLoader` overload is deprecated for removal and is no longer called — a subclass overriding it has to move to the new signature. `Serialization` is an internal utility and `PersistentObjectInputStream` is the only in-tree override; shout if you would rather keep the old hook consulted for compatibility. No behaviour change otherwise: loader order, de-duplication, null filtering and the fallback to the stream own `resolveClass` are untouched, and `TestSerializationClassResolving` (4 tests) stays green, as does the kernel module. `TestDetachedStateManager` errors 10/10 both here and on unmodified master (`CREATE TABLE SEC1_TAB ()`), so it is pre-existing and unrelated. -- 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]
