Make CursorStrategy and default ScrollSubrange nullable

See gh-597

Author: rstoyanchev

spring-graphql/src/main/java/org/springframework/graphql/data/query/QueryByExampleDataFetcher.java

@@ -195,18 +195,15 @@ public static <T> ReactiveBuilder<T, T> builder(ReactiveQueryByExampleExecutor<T
 
 	/**
 	 * Variation of {@link #autoRegistrationConfigurer(List, List, CursorStrategy, ScrollSubrange)}
-	 * that defaults to the following:
-	 * <ul>
-	 * <li>{@link ScrollPositionCursorStrategy} with Base64 encoding.
-	 * <li>{@link OffsetScrollPosition Offset}-based scrolling with 20 items at a time
-	 * </ul>
+	 * without a {@code CursorStrategy} and default {@link ScrollSubrange}.
+	 * For default values, see the respective methods on {@link Builder} and
+	 * {@link ReactiveBuilder}.
 	 */
 	public static RuntimeWiringConfigurer autoRegistrationConfigurer(
 			List<QueryByExampleExecutor<?>> executors,
 			List<ReactiveQueryByExampleExecutor<?>> reactiveExecutors) {
 
-		return autoRegistrationConfigurer(executors, reactiveExecutors,
-				RepositoryUtils.defaultCursorStrategy(), RepositoryUtils.defaultScrollSubrange());
+		return autoRegistrationConfigurer(executors, reactiveExecutors, null, null);
 	}
 
 	/**
@@ -220,16 +217,18 @@ public static RuntimeWiringConfigurer autoRegistrationConfigurer(
 	 *
 	 * @param executors repositories to consider for registration
 	 * @param reactiveExecutors reactive repositories to consider for registration
-	 * @param cursorStrategy for decoding cursors in pagination requests
-	 * @param defaultScrollSubrange default parameters for scrolling
+	 * @param cursorStrategy for decoding cursors in pagination requests;
+	 * if {@code null}, then {@link Builder#cursorStrategy} defaults apply.
+	 * @param defaultScrollSubrange default parameters for scrolling;
+	 * if {@code null}, then {@link Builder#defaultScrollSubrange} defaults apply.
 	 * @return the created configurer
 	 * @since 1.2
 	 */
 	public static RuntimeWiringConfigurer autoRegistrationConfigurer(
 			List<QueryByExampleExecutor<?>> executors,
 			List<ReactiveQueryByExampleExecutor<?>> reactiveExecutors,
-			CursorStrategy<ScrollPosition> cursorStrategy,
-			ScrollSubrange defaultScrollSubrange) {
+			@Nullable CursorStrategy<ScrollPosition> cursorStrategy,
+			@Nullable ScrollSubrange defaultScrollSubrange) {
 
 		Map<String, DataFetcherFactory> factories = new HashMap<>();
 

spring-graphql/src/main/java/org/springframework/graphql/data/query/QuerydslDataFetcher.java

@@ -219,18 +219,15 @@ public static <T> ReactiveBuilder<T, T> builder(ReactiveQuerydslPredicateExecuto
 
 	/**
 	 * Variation of {@link #autoRegistrationConfigurer(List, List, CursorStrategy, ScrollSubrange)}
-	 * that defaults to the following:
-	 * <ul>
-	 * <li>{@link ScrollPositionCursorStrategy} with Base64 encoding.
-	 * <li>{@link OffsetScrollPosition Offset}-based scrolling with 20 items at a time
-	 * </ul>
+	 * without a {@code CursorStrategy} and default {@link ScrollSubrange}.
+	 * For default values, see the respective methods on {@link Builder} and
+	 * {@link ReactiveBuilder}.
 	 */
 	public static RuntimeWiringConfigurer autoRegistrationConfigurer(
 			List<QuerydslPredicateExecutor<?>> executors,
 			List<ReactiveQuerydslPredicateExecutor<?>> reactiveExecutors) {
 
-		return autoRegistrationConfigurer(executors, reactiveExecutors,
-				RepositoryUtils.defaultCursorStrategy(), RepositoryUtils.defaultScrollSubrange());
+		return autoRegistrationConfigurer(executors, reactiveExecutors, null, null);
 	}
 
 	/**
@@ -247,15 +244,19 @@ public static RuntimeWiringConfigurer autoRegistrationConfigurer(
 	 *
 	 * @param executors repositories to consider for registration
 	 * @param reactiveExecutors reactive repositories to consider for registration
+	 * @param cursorStrategy for decoding cursors in pagination requests;
+	 * if {@code null}, then {@link Builder#cursorStrategy} defaults apply.
+	 * @param defaultScrollSubrange default parameters for scrolling;
+	 * if {@code null}, then {@link Builder#defaultScrollSubrange} defaults apply.
 	 * @return the created configurer
 	 * @since 1.2
 	 */
 	@SuppressWarnings({"unchecked", "rawtypes"})
 	public static RuntimeWiringConfigurer autoRegistrationConfigurer(
 			List<QuerydslPredicateExecutor<?>> executors,
 			List<ReactiveQuerydslPredicateExecutor<?>> reactiveExecutors,
-			CursorStrategy<ScrollPosition> cursorStrategy,
-			ScrollSubrange defaultScrollSubrange) {
+			@Nullable CursorStrategy<ScrollPosition> cursorStrategy,
+			@Nullable ScrollSubrange defaultScrollSubrange) {
 
 		Map<String, DataFetcherFactory> factories = new HashMap<>();