Polish

- Polish javadoc
- Make ReindexerQueryExecutionConverters package private
- Add since tag

Author: evgeniycheban

src/main/java/org/springframework/data/reindexer/repository/support/ReindexerAnnotationRepositoryMetadata.java

@@ -17,7 +17,6 @@
 
 import java.lang.reflect.Method;
 
-import org.springframework.data.reindexer.repository.util.ReindexerQueryExecutionConverters;
 import org.springframework.data.repository.core.support.AnnotationRepositoryMetadata;
 import org.springframework.data.repository.util.ReactiveWrapperConverters;
 import org.springframework.data.util.TypeInformation;

src/main/java/org/springframework/data/reindexer/repository/support/ReindexerDefaultRepositoryMetadata.java

@@ -17,7 +17,6 @@
 
 import java.lang.reflect.Method;
 
-import org.springframework.data.reindexer.repository.util.ReindexerQueryExecutionConverters;
 import org.springframework.data.repository.core.support.DefaultRepositoryMetadata;
 import org.springframework.data.repository.util.ReactiveWrapperConverters;
 import org.springframework.data.util.TypeInformation;

src/main/java/org/springframework/data/reindexer/repository/util/ReindexerQueryExecutionConverters.java renamed to src/main/java/org/springframework/data/reindexer/repository/support/ReindexerQueryExecutionConverters.java

@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package org.springframework.data.reindexer.repository.util;
+package org.springframework.data.reindexer.repository.support;
 
 import java.util.stream.Stream;
 
@@ -26,28 +26,16 @@
 import org.springframework.util.Assert;
 
 /**
- * Converters to potentially wrap the execution of a repository method into a variety of wrapper types
- * potentially being available on the classpath.
- * @see QueryExecutionConverters
+ * For internal use only, as this contract is likely to change.
  *
  * @author Evgeniy Cheban
  */
-public final class ReindexerQueryExecutionConverters {
+final class ReindexerQueryExecutionConverters {
 
 	private ReindexerQueryExecutionConverters() {
 	}
 
-	/**
-	 * Recursively unwraps well known wrapper types from the given {@link TypeInformation} but aborts at the given
-	 * reference type.
-	 * This method is a copy of {@link QueryExecutionConverters#unwrapWrapperTypes(TypeInformation, TypeInformation)}
-	 * with extension of adding {@link ResultIterator} type check.
-	 *
-	 * @param type must not be {@literal null}.
-	 * @param reference must not be {@literal null}.
-	 * @return will never be {@literal null}.
-	 */
-	public static TypeInformation<?> unwrapWrapperTypes(TypeInformation<?> type, TypeInformation<?> reference) {
+	static TypeInformation<?> unwrapWrapperTypes(TypeInformation<?> type, TypeInformation<?> reference) {
 		Assert.notNull(type, "type must not be null");
 		if (reference.isAssignableFrom(type)) {
 			return type;
@@ -62,5 +50,4 @@ public static TypeInformation<?> unwrapWrapperTypes(TypeInformation<?> type, Typ
 				|| ResultIterator.class.isAssignableFrom(rawType);
 		return needToUnwrap ? unwrapWrapperTypes(type.getRequiredComponentType(), reference) : type;
 	}
-
 }

src/main/java/org/springframework/data/reindexer/repository/util/PageableUtils.java

@@ -19,10 +19,11 @@
 import org.springframework.data.domain.Pageable;
 
 /**
- * Provide a set of utility methods to support {@link Pageable}s.
+ * Provides a set of utility methods to support {@link Pageable}s.
  *
  * @author Greg Turnquist
  * @author Evgeniy Cheban
+ * @since 1.3
  */
 public final class PageableUtils {
 
@@ -31,7 +32,7 @@ private PageableUtils() {
 	}
 
 	/**
-	 * Convert a {@link Pageable}'s offset value from {@link Long} to {@link Integer} to support JPA spec methods.
+	 * Converts a {@link Pageable}'s offset value from {@link Long} to {@link Integer}.
 	 *
 	 * @param pageable the {@link Pageable} to use
 	 * @return integer