HHH-18875 Remove the related GraalVM reflective registrations

hibernate · Nov 22, 2024 · c3419d2 · c3419d2
1 parent 25fd2dd
commit c3419d2
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 57 deletions.
diff --git a/hibernate-graalvm/src/main/java/org/hibernate/graalvm/internal/StaticClassLists.java b/hibernate-graalvm/src/main/java/org/hibernate/graalvm/internal/StaticClassLists.java
@@ -4,24 +4,7 @@
  */
 package org.hibernate.graalvm.internal;
 
-import java.util.List;
-import java.util.Map;
-import java.util.function.Function;
-import java.util.function.Supplier;
 
-import org.hibernate.graph.internal.parse.SubGraphGenerator;
-import org.hibernate.graph.spi.AttributeNodeImplementor;
-import org.hibernate.graph.spi.GraphImplementor;
-import org.hibernate.query.hql.spi.DotIdentifierConsumer;
-import org.hibernate.query.hql.spi.SqmCreationProcessingState;
-import org.hibernate.query.sqm.spi.ParameterDeclarationContext;
-import org.hibernate.query.sqm.sql.FromClauseIndex;
-import org.hibernate.sql.ast.Clause;
-import org.hibernate.sql.ast.spi.SqlAstProcessingState;
-import org.hibernate.sql.ast.tree.Statement;
-import org.hibernate.sql.ast.tree.select.QueryPart;
-import org.hibernate.sql.results.graph.FetchParent;
-import org.hibernate.sql.results.jdbc.spi.JdbcValuesSourceProcessingState;
 import org.hibernate.tool.schema.internal.script.MultiLineSqlScriptExtractor;
 import org.hibernate.type.EnumType;
 
@@ -94,25 +77,6 @@ public static Class[] typesNeedingArrayCopy() {
 				org.hibernate.event.spi.PostCollectionRecreateEventListener[].class,
 				org.hibernate.event.spi.PostCollectionRemoveEventListener[].class,
 				org.hibernate.event.spi.PostCollectionUpdateEventListener[].class,
-				//And other array types, necessary for allocation of generified instances of org.hibernate.internal.util.collections.StandardStack:
-				//TODO can this list be tested for consistency with the core module? Or generated? e.g. could use Jandex?
-				AttributeNodeImplementor[].class,
-				Clause[].class,
-				DotIdentifierConsumer[].class,
-				FetchParent[].class,
-				FromClauseIndex[].class,
-				Function[].class,
-				GraphImplementor[].class,
-				JdbcValuesSourceProcessingState[].class,
-				List[].class,
-				Map.Entry[].class,
-				ParameterDeclarationContext[].class,
-				QueryPart[].class,
-				SqlAstProcessingState[].class,
-				SqmCreationProcessingState[].class,
-				Statement[].class,
-				SubGraphGenerator[].class,
-				Supplier[].class,
 		};
 	}
 

diff --git a/hibernate-graalvm/src/test/java/org/hibernate/graalvm/internal/StaticClassListsTest.java b/hibernate-graalvm/src/test/java/org/hibernate/graalvm/internal/StaticClassListsTest.java
@@ -158,27 +158,7 @@ Stream<Class<?>> classes() {
 				// Putting anything here is running the risk of forgetting
 				// why it was necessary in the first place...
 				return Stream.of(
-						// Java classes -- the why is lost to history
-						java.util.function.Function[].class,
-						java.util.List[].class,
-						java.util.Map.Entry[].class,
-						java.util.function.Supplier[].class,
-						// Graphs -- the why is lost to history
-						org.hibernate.graph.spi.AttributeNodeImplementor[].class,
-						org.hibernate.sql.results.graph.FetchParent[].class,
-						org.hibernate.graph.spi.GraphImplementor[].class,
-						org.hibernate.graph.internal.parse.SubGraphGenerator[].class,
-						// AST/parsing -- no way to detect this automatically, you just have to know.
-						org.hibernate.sql.ast.Clause[].class,
-						org.hibernate.query.hql.spi.DotIdentifierConsumer[].class,
-						org.hibernate.query.sqm.sql.FromClauseIndex[].class,
-						org.hibernate.query.sqm.spi.ParameterDeclarationContext[].class,
-						org.hibernate.sql.ast.tree.select.QueryPart[].class,
-						org.hibernate.sql.ast.spi.SqlAstProcessingState[].class,
-						org.hibernate.query.hql.spi.SqmCreationProcessingState[].class,
-						org.hibernate.sql.ast.tree.Statement[].class,
-						// Various internals -- the why is lost to history
-						org.hibernate.sql.results.jdbc.spi.JdbcValuesSourceProcessingState[].class
+						// Hopefully to remain empty
 				);
 			}
 		};