HHH-18976 Register types requiring Arrays#copyOf in org.hibernate.graalvm.internal.StaticClassLists

Author: yrodiere

hibernate-core/src/main/java/org/hibernate/bytecode/enhance/internal/tracker/CompositeOwnerTracker.java

@@ -47,7 +47,7 @@ public void callOwner(String fieldName) {
 		}
 	}
 
-	@AllowReflection
+	@AllowReflection // Possible array types are registered in org.hibernate.graalvm.internal.StaticClassLists.typesNeedingArrayCopy
 	public void removeOwner(String name) {
 		for ( int i = 0; i < names.length; i++ ) {
 			if ( name.equals( names[i] ) ) {

hibernate-core/src/main/java/org/hibernate/engine/internal/SessionEventListenerManagerImpl.java

@@ -31,7 +31,7 @@ public SessionEventListenerManagerImpl(List<SessionEventListener> initialListene
 	}
 
 	@Override
-	@AllowReflection
+	@AllowReflection // Possible array types are registered in org.hibernate.graalvm.internal.StaticClassLists.typesNeedingArrayCopy
 	public void addListener(final SessionEventListener... additionalListeners) {
 		Objects.requireNonNull( additionalListeners );
 		final SessionEventListener[] existing = this.listeners;

hibernate-graalvm/src/main/java/org/hibernate/graalvm/internal/StaticClassLists.java

@@ -5,6 +5,8 @@
 package org.hibernate.graalvm.internal;
 
 
+import org.hibernate.SessionEventListener;
+import org.hibernate.engine.spi.CompositeTracker;
 import org.hibernate.tool.schema.internal.script.MultiLineSqlScriptExtractor;
 
 /**
@@ -41,6 +43,10 @@ public static Class[] typesNeedingDefaultConstructorAccessible() {
 
 	public static Class[] typesNeedingArrayCopy() {
 		return new Class[] {
+				// See CompositeOwnerTracker
+				CompositeTracker[].class,
+				// See SessionEventListenerManagerImpl
+				SessionEventListener[].class,
 				//Eventlisteners need to be registered for reflection to allow creation via Array#newInstance ;
 				// types need to be in synch with those declared in org.hibernate.event.spi.EventType
 				org.hibernate.event.spi.LoadEventListener[].class,

hibernate-graalvm/src/test/java/org/hibernate/graalvm/internal/StaticClassListsTest.java

@@ -15,6 +15,8 @@
 import java.util.stream.Stream;
 
 import org.hibernate.Session;
+import org.hibernate.SessionEventListener;
+import org.hibernate.engine.spi.CompositeOwner;
 import org.hibernate.event.spi.EventType;
 import org.hibernate.internal.util.ReflectHelper;
 import org.hibernate.persister.collection.CollectionPersister;
@@ -56,6 +58,16 @@ void meta_noMissingTestCategory() {
 
 	// TODO ORM 7: Move this inside TypesNeedingAllConstructorsAccessible (requires JDK 17) and rename to simple Category
 	enum TypesNeedingAllConstructorsAccessible_Category {
+		// These are needed simply because we use Arrays#copyOf with arrays of this type
+		ARRAYS_COPY_OF {
+			@Override
+			Stream<Class<?>> classes() {
+				return Stream.of(
+						CompositeOwner.class, // See org.hibernate.bytecode.enhance.internal.tracker.CompositeOwnerTracker
+						SessionEventListener.class // See org.hibernate.engine.internal.SessionEventListenerManagerImpl
+				);
+			}
+		},
 		PERSISTERS {
 			@Override
 			Stream<Class<?>> classes() {