HHH-18863 change the way enum types -> values are cached to avoid lookups

Author: gavinking

tooling/metamodel-generator/src/main/java/org/hibernate/processor/Context.java

@@ -412,15 +412,15 @@ public boolean checkNamedQuery(String name) {
 	public void setUsesQuarkusOrm(boolean b) {
 		usesQuarkusOrm = b;
 	}
-	
+
 	public boolean usesQuarkusOrm() {
 		return usesQuarkusOrm;
 	}
 
 	public void setUsesQuarkusReactive(boolean b) {
 		usesQuarkusReactive = b;
 	}
-	
+
 	public boolean usesQuarkusReactive() {
 		return usesQuarkusReactive;
 	}
@@ -504,7 +504,21 @@ public Map<String,Set<String>> getEnumTypesByValue() {
 		return enumTypesByValue;
 	}
 
-	public void addEnumValue(String type, String value) {
-		enumTypesByValue.computeIfAbsent( value, s -> new TreeSet<>() ).add( type );
+	public void addEnumValue(
+			String qualifiedTypeName, String shortTypeName,
+			@Nullable String outerTypeQualifiedName, @Nullable String outerShortTypeName,
+			String value) {
+		addEnumValue( qualifiedTypeName, value );
+		addEnumValue( qualifiedTypeName, qualifiedTypeName + '.' + value );
+		addEnumValue( qualifiedTypeName, shortTypeName + '.' + value );
+		if ( outerShortTypeName != null ) {
+			addEnumValue( qualifiedTypeName, outerShortTypeName + '.' + shortTypeName + '.' + value );
+			addEnumValue( qualifiedTypeName, outerShortTypeName + '$' + shortTypeName + '.' + value );
+			addEnumValue( qualifiedTypeName, outerTypeQualifiedName + '$' + shortTypeName + '.' + value );
+		}
+	}
+
+	private void addEnumValue(String qualifiedTypeName, String value) {
+		enumTypesByValue.computeIfAbsent( value, s -> new TreeSet<>() ).add( qualifiedTypeName );
 	}
 }

tooling/metamodel-generator/src/main/java/org/hibernate/processor/HibernateProcessor.java

@@ -646,9 +646,15 @@ private void indexEnumValues(TypeMirror type) {
 			final DeclaredType declaredType = (DeclaredType) type;
 			final TypeElement fieldType = (TypeElement) declaredType.asElement();
 			if ( fieldType.getKind() == ElementKind.ENUM ) {
-				for  (Element enumMember : fieldType.getEnclosedElements() ) {
+				for ( Element enumMember : fieldType.getEnclosedElements() ) {
 					if ( enumMember.getKind() == ElementKind.ENUM_CONSTANT ) {
+						final Element enclosingElement = fieldType.getEnclosingElement();
+						final boolean hasOuterType =
+								enclosingElement.getKind().isClass() || enclosingElement.getKind().isInterface();
 						context.addEnumValue( fieldType.getQualifiedName().toString(),
+								fieldType.getSimpleName().toString(),
+								hasOuterType ? ((TypeElement) enclosingElement).getQualifiedName().toString() : null,
+								hasOuterType ? enclosingElement.getSimpleName().toString() : null,
 								enumMember.getSimpleName().toString() );
 					}
 				}

tooling/metamodel-generator/src/main/java/org/hibernate/processor/validation/ProcessorSessionFactory.java

@@ -59,7 +59,6 @@
 import static org.hibernate.internal.util.StringHelper.root;
 import static org.hibernate.internal.util.StringHelper.split;
 import static org.hibernate.internal.util.StringHelper.unroot;
-import static org.hibernate.metamodel.model.domain.internal.JpaMetamodelImpl.addAllowedEnumLiteralsToEnumTypesMap;
 import static org.hibernate.processor.util.Constants.JAVA_OBJECT;
 
 /**
@@ -92,7 +91,7 @@ public static MockSessionFactory create(
 	private final Types typeUtil;
 	private final Filer filer;
 	private final Map<String, String> entityNameMappings;
-	private final Map<String, Set<String>> allowedEnumLiteralsToEnumTypeNames;
+	private final Map<String, Set<String>> enumTypesByValue;
 
 	public ProcessorSessionFactory(
 			ProcessingEnvironment processingEnvironment,
@@ -102,23 +101,7 @@ public ProcessorSessionFactory(
 		typeUtil = processingEnvironment.getTypeUtils();
 		filer = processingEnvironment.getFiler();
 		this.entityNameMappings = entityNameMappings;
-		final Map<String, Set<String>> allowedEnumLiteralsToEnumTypeNames = new HashMap<>( enumTypesByValue.size() << 2 );
-		for ( Map.Entry<String, Set<String>> entry : enumTypesByValue.entrySet() ) {
-			final String enumConstantName = entry.getKey();
-			for ( String enumClassName : entry.getValue() ) {
-				final TypeElement enumTypeElement = elementUtil.getTypeElement( enumClassName );
-				if ( enumTypeElement != null ) {
-					addAllowedEnumLiteralsToEnumTypesMap(
-							allowedEnumLiteralsToEnumTypeNames,
-							enumConstantName,
-							enumTypeElement.getSimpleName().toString(),
-							elementUtil.getBinaryName( enumTypeElement ).toString(),
-							enumClassName
-					);
-				}
-			}
-		}
-		this.allowedEnumLiteralsToEnumTypeNames = allowedEnumLiteralsToEnumTypeNames;
+		this.enumTypesByValue = enumTypesByValue;
 	}
 
 	@Override
@@ -236,7 +219,7 @@ private static JdbcType enumJdbcType(Element member) {
 
 	@Override @Nullable
 	Set<String> getEnumTypesForValue(String value) {
-		Set<String> result = allowedEnumLiteralsToEnumTypeNames.get( value);
+		Set<String> result = enumTypesByValue.get(value);
 		if ( result != null ) {
 			return result;
 		}