HHH-19918 Avoid reflection when instantiating known FormatMapper

Author: beikov

hibernate-core/src/main/java/org/hibernate/type/format/jackson/JacksonIntegration.java

@@ -41,67 +41,38 @@ private static boolean ableToLoadJacksonOSONFactory() {
 
 	public static @Nullable FormatMapper getXMLJacksonFormatMapperOrNull(FormatMapperCreationContext creationContext) {
 		return JACKSON_XML_AVAILABLE
-				? createFormatMapper( "org.hibernate.type.format.jackson.JacksonXmlFormatMapper", creationContext )
+				? new JacksonXmlFormatMapper( creationContext )
 				: null;
 	}
 
 	public static @Nullable FormatMapper getJsonJacksonFormatMapperOrNull(FormatMapperCreationContext creationContext) {
 		return JACKSON_JSON_AVAILABLE
-				? createFormatMapper( "org.hibernate.type.format.jackson.JacksonJsonFormatMapper", creationContext )
+				? new JacksonJsonFormatMapper( creationContext )
 				: null;
 	}
 	public static @Nullable FormatMapper getOsonJacksonFormatMapperOrNull(FormatMapperCreationContext creationContext) {
 		return JACKSON_OSON_AVAILABLE
-				? createFormatMapper( "org.hibernate.type.format.jackson.JacksonOsonFormatMapper", creationContext )
+				? new JacksonOsonFormatMapper( creationContext )
 				: null;
 	}
 
 	public static @Nullable FormatMapper getXMLJacksonFormatMapperOrNull(boolean legacyFormat) {
-		if ( JACKSON_XML_AVAILABLE ) {
-			try {
-				final Class<?> formatMapperClass = JacksonIntegration.class.getClassLoader()
-						.loadClass( "org.hibernate.type.format.jackson.JacksonXmlFormatMapper" );
-				return (FormatMapper) formatMapperClass.getDeclaredConstructor( boolean.class )
-						.newInstance( legacyFormat );
-			}
-			catch (Exception e) {
-				throw new RuntimeException( "Couldn't instantiate Jackson XML FormatMapper", e );
-			}
-		}
-		return null;
+		return JACKSON_XML_AVAILABLE
+				? new JacksonXmlFormatMapper( legacyFormat )
+				: null;
 	}
 
 	public static @Nullable FormatMapper getJsonJacksonFormatMapperOrNull() {
 		return JACKSON_JSON_AVAILABLE
-				? createFormatMapper( "org.hibernate.type.format.jackson.JacksonJsonFormatMapper", null )
+				? new JacksonJsonFormatMapper()
 				: null;
 	}
 	public static @Nullable FormatMapper getOsonJacksonFormatMapperOrNull() {
 		return JACKSON_OSON_AVAILABLE
-				? createFormatMapper( "org.hibernate.type.format.jackson.JacksonOsonFormatMapper", null )
+				? new JacksonOsonFormatMapper()
 				: null;
 	}
 
-	private static FormatMapper createFormatMapper(String className, @Nullable FormatMapperCreationContext creationContext) {
-		try {
-			if ( creationContext == null ) {
-				final Class<?> formatMapperClass = JacksonIntegration.class.getClassLoader()
-						.loadClass( className );
-				return (FormatMapper) formatMapperClass.getDeclaredConstructor().newInstance();
-			}
-			else {
-				return (FormatMapper) creationContext.getBootstrapContext()
-						.getClassLoaderAccess()
-						.classForName( className )
-						.getDeclaredConstructor( FormatMapperCreationContext.class )
-						.newInstance( creationContext );
-			}
-		}
-		catch (Exception e) {
-			throw new RuntimeException( "Couldn't instantiate Jackson FormatMapper", e );
-		}
-	}
-
 	/**
 	 * Checks that Oracle OSON extension available
 	 *