HHH-19918 Avoid reflection when instantiating known FormatMapper

Author: beikov

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

@@ -42,57 +42,22 @@ 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 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;
-	}
-
 	public static @Nullable FormatMapper getJsonJacksonFormatMapperOrNull() {
 		return JACKSON_JSON_AVAILABLE
-				? createFormatMapper( "org.hibernate.type.format.jackson.JacksonJsonFormatMapper", null )
+				? new JacksonJsonFormatMapper()
 				: 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 );
-		}
-	}
-
 	private static boolean canLoad(String name) {
 		try {
 			//N.B. intentionally not using the context classloader