Polishing.

Original Pull Request: #577

Author: christophstrobl

src/main/antora/modules/ROOT/pages/keyvalue/repository/map-repositories.adoc

@@ -2,7 +2,7 @@
 = Map Repositories
 
 Map repositories reside on top of the `KeyValueTemplate`.
-Using the default `SpelQueryCreator` allows deriving query and sort expressions from the given method name, as the following example shows:
+Using the default `PredicateQueryCreator` allows deriving query and sort expressions from the given method name, as the following example shows:
 
 [source, java]
 ----
@@ -16,3 +16,10 @@ interface PersonRepository implements CrudRepository<Person, String> {
     List<Person> findByLastname(String lastname);
 }
 ----
+
+== Configuring the QueryEngine
+
+It is possible to change the `QueryEngine` and use a custom one instead of the default.
+The `EnableMapRepositories` annotation allows to configure the by supplying a `QueryEngineFactory` as well as the `QueryCreator` via according attributes.
+Please mind that the `QueryEngine` needs to be able to process queries created by the configured `QueryCreator`.
+

src/main/java/org/springframework/data/keyvalue/repository/query/SpelQueryCreator.java

@@ -181,7 +181,7 @@ protected SpelExpression toPredicateExpression(PartTree tree) {
 					case EXISTS:
 					default:
 						throw new InvalidDataAccessApiUsageException(
-								String.format("Found invalid part '%s' in query", part.getType()));
+								"Found invalid part '%s' in query".formatted(part.getType()));
 				}
 
 				if (partIter.hasNext()) {

src/main/java/org/springframework/data/map/repository/config/MapRepositoryConfigurationExtension.java

@@ -109,20 +109,19 @@ private static SortAccessor<?> getSortAccessor(RepositoryConfigurationSource sou
 		Class<? extends QueryEngineFactory> queryEngineFactoryType = (Class<? extends QueryEngineFactory>) getAnnotationAttributes(
 				source).get("queryEngineFactory");
 
-		if (queryEngineFactoryType != null && !queryEngineFactoryType.isInterface()) {
-
-			if (sortAccessor != null) {
-				Constructor<? extends QueryEngineFactory> constructor = ClassUtils
-						.getConstructorIfAvailable(queryEngineFactoryType, SortAccessor.class);
-				if (constructor != null) {
-					return BeanUtils.instantiateClass(constructor, sortAccessor).create();
-				}
-			}
+		if(queryEngineFactoryType == null || queryEngineFactoryType.isInterface()) {
+			return null;
+		}
 
-			return BeanUtils.instantiateClass(queryEngineFactoryType).create();
+		if (sortAccessor != null) {
+			Constructor<? extends QueryEngineFactory> constructor = ClassUtils
+					.getConstructorIfAvailable(queryEngineFactoryType, SortAccessor.class);
+			if (constructor != null) {
+				return BeanUtils.instantiateClass(constructor, sortAccessor).create();
+			}
 		}
 
-		return null;
+		return BeanUtils.instantiateClass(queryEngineFactoryType).create();
 	}
 
 	private static Map<String, Object> getAnnotationAttributes(RepositoryConfigurationSource source) {