Eagerly initialize SimplePropertyValueConversions.

Closes #2590
Original pull request: #2591.

Author: christophstrobl

src/main/java/org/springframework/data/convert/CustomConversions.java

@@ -947,7 +947,7 @@ public ConverterConfiguration(StoreConversions storeConversions, List<?> userCon
 		public ConverterConfiguration(StoreConversions storeConversions, List<?> userConverters,
 				Predicate<ConvertiblePair> converterRegistrationFilter) {
 
-			this(storeConversions, userConverters, converterRegistrationFilter, new SimplePropertyValueConversions());
+			this(storeConversions, userConverters, converterRegistrationFilter, PropertyValueConversions.simple(it -> {}));
 		}
 
 		/**

src/main/java/org/springframework/data/convert/PropertyValueConversions.java

@@ -63,6 +63,11 @@ static <P extends PersistentProperty<P>> PropertyValueConversions simple(
 		PropertyValueConverterRegistrar registrar = new PropertyValueConverterRegistrar();
 		config.accept(registrar);
 		conversions.setValueConverterRegistry(registrar.buildRegistry());
+		try {
+			conversions.afterPropertiesSet();
+		} catch (Exception e) {
+			throw new IllegalStateException("Could not initialize value conversions.");
+		}
 		return conversions;
 	}
 }

src/main/java/org/springframework/data/convert/SimplePropertyValueConversions.java

@@ -17,7 +17,6 @@
 
 import java.util.ArrayList;
 import java.util.List;
-import java.util.concurrent.atomic.AtomicBoolean;
 
 import org.springframework.beans.factory.InitializingBean;
 import org.springframework.data.convert.PropertyValueConverterFactories.ChainedPropertyValueConverterFactory;
@@ -40,7 +39,6 @@ public class SimplePropertyValueConversions implements PropertyValueConversions,
 	private @Nullable PropertyValueConverterFactory converterFactory;
 	private @Nullable ValueConverterRegistry<?> valueConverterRegistry;
 	private boolean converterCacheEnabled = true;
-	private final AtomicBoolean initialized = new AtomicBoolean(false);
 
 	/**
 	 * Set the {@link PropertyValueConverterFactory factory} responsible for creating the actual
@@ -91,23 +89,13 @@ public void setConverterCacheEnabled(boolean converterCacheEnabled) {
 
 	@Override
 	public boolean hasValueConverter(PersistentProperty<?> property) {
-
-		if (!initialized.get()) {
-			init();
-		}
-
 		return this.converterFactory.getConverter(property) != null;
 	}
 
 	@Nullable
 	@Override
 	public <DV, SV, C extends PersistentProperty<C>, D extends ValueConversionContext<C>> PropertyValueConverter<DV, SV, D> getValueConverter(
 			C property) {
-
-		if (!initialized.get()) {
-			init();
-		}
-
 		return this.converterFactory.getConverter(property);
 	}
 
@@ -116,27 +104,24 @@ public <DV, SV, C extends PersistentProperty<C>, D extends ValueConversionContex
 	 */
 	public void init() {
 
-		if (initialized.compareAndSet(false, true)) {
-
-			List<PropertyValueConverterFactory> factoryList = new ArrayList<>(3);
+		List<PropertyValueConverterFactory> factoryList = new ArrayList<>(3);
 
-			if (converterFactory != null) {
-				factoryList.add(converterFactory);
-			} else {
-				factoryList.add(PropertyValueConverterFactory.simple());
-			}
+		if (converterFactory != null) {
+			factoryList.add(converterFactory);
+		} else {
+			factoryList.add(PropertyValueConverterFactory.simple());
+		}
 
-			if ((valueConverterRegistry != null) && !valueConverterRegistry.isEmpty()) {
-				factoryList.add(PropertyValueConverterFactory.configuredInstance(valueConverterRegistry));
-			}
+		if ((valueConverterRegistry != null) && !valueConverterRegistry.isEmpty()) {
+			factoryList.add(PropertyValueConverterFactory.configuredInstance(valueConverterRegistry));
+		}
 
-			PropertyValueConverterFactory targetFactory = factoryList.size() > 1
-					? PropertyValueConverterFactory.chained(factoryList)
-					: factoryList.iterator().next();
+		PropertyValueConverterFactory targetFactory = factoryList.size() > 1
+				? PropertyValueConverterFactory.chained(factoryList)
+				: factoryList.iterator().next();
 
-			this.converterFactory = converterCacheEnabled ? PropertyValueConverterFactory.caching(targetFactory)
-					: targetFactory;
-		}
+		this.converterFactory = converterCacheEnabled ? PropertyValueConverterFactory.caching(targetFactory)
+				: targetFactory;
 	}
 
 	@Override