Add ByteUtils utility static class for byte array/string conversions

Re-order author tags correctly
Remove excessive final modifiers

Signed-off-by: Emilien Bevierre <emilien.bevierre@couchbase.com>

Author: emilienbev

src/main/java/org/springframework/data/couchbase/cache/CouchbaseCacheConfiguration.java

@@ -16,7 +16,7 @@
 
 package org.springframework.data.couchbase.cache;
 
-import java.nio.charset.StandardCharsets;
+import org.springframework.data.couchbase.core.util.ByteUtils;
 import java.time.Duration;
 
 import org.springframework.cache.Cache;
@@ -60,7 +60,8 @@ public static CouchbaseCacheConfiguration defaultCacheConfig() {
 	}
 
 	/**
-	 * Registers default cache key converters. The following converters get registered:
+	 * Registers default cache key converters. The following converters get
+	 * registered:
 	 * <ul>
 	 * <li>{@link String} to byte using UTF-8 encoding.</li>
 	 * <li>{@link SimpleKey} to {@link String}</li>
@@ -70,12 +71,13 @@ public static CouchbaseCacheConfiguration defaultCacheConfig() {
 	 */
 	public static void registerDefaultConverters(final ConverterRegistry registry) {
 		Assert.notNull(registry, "ConverterRegistry must not be null!");
-		registry.addConverter(String.class, byte[].class, source -> source.getBytes(StandardCharsets.UTF_8));
+		registry.addConverter(String.class, byte[].class, ByteUtils::getBytes);
 		registry.addConverter(SimpleKey.class, String.class, SimpleKey::toString);
 	}
 
 	/**
-	 * Set the expiry to apply for cache entries. Use {@link Duration#ZERO} to declare an eternal cache.
+	 * Set the expiry to apply for cache entries. Use {@link Duration#ZERO} to
+	 * declare an eternal cache.
 	 *
 	 * @param expiry must not be {@literal null}.
 	 * @return new {@link CouchbaseCacheConfiguration}.
@@ -112,9 +114,13 @@ public CouchbaseCacheConfiguration valueTranscoder(final Transcoder valueTransco
 
 	/**
 	 * Disable caching {@literal null} values. <br />
-	 * <strong>NOTE</strong> any {@link org.springframework.cache.Cache#put(Object, Object)} operation involving
-	 * {@literal null} value will error. Nothing will be written to Couchbase, nothing will be removed. An already
-	 * existing key will still be there afterwards with the very same value as before.
+	 * <strong>NOTE</strong> any
+	 * {@link org.springframework.cache.Cache#put(Object, Object)} operation
+	 * involving
+	 * {@literal null} value will error. Nothing will be written to Couchbase,
+	 * nothing will be removed. An already
+	 * existing key will still be there afterwards with the very same value as
+	 * before.
 	 *
 	 * @return new {@link CouchbaseCacheConfiguration}.
 	 */
@@ -124,8 +130,10 @@ public CouchbaseCacheConfiguration disableCachingNullValues() {
 	}
 
 	/**
-	 * Prefix the {@link CouchbaseCache#getName() cache name} with the given value. <br />
-	 * The generated cache key will be: {@code prefix + cache name + "::" + cache entry key}.
+	 * Prefix the {@link CouchbaseCache#getName() cache name} with the given value.
+	 * <br />
+	 * The generated cache key will be:
+	 * {@code prefix + cache name + "::" + cache entry key}.
 	 *
 	 * @param prefix the prefix to prepend to the cache name.
 	 * @return this.
@@ -137,7 +145,8 @@ public CouchbaseCacheConfiguration prefixCacheNameWith(final String prefix) {
 	}
 
 	/**
-	 * Use the given {@link CacheKeyPrefix} to compute the prefix for the actual Couchbase {@literal key} given the
+	 * Use the given {@link CacheKeyPrefix} to compute the prefix for the actual
+	 * Couchbase {@literal key} given the
 	 * {@literal cache name} as function input.
 	 *
 	 * @param cacheKeyPrefix must not be {@literal null}.
@@ -165,14 +174,16 @@ public boolean getAllowCacheNullValues() {
 	}
 
 	/**
-	 * @return The {@link ConversionService} used for cache key to {@link String} conversion. Never {@literal null}.
+	 * @return The {@link ConversionService} used for cache key to {@link String}
+	 *         conversion. Never {@literal null}.
 	 */
 	public ConversionService getConversionService() {
 		return conversionService;
 	}
 
 	/**
-	 * @return {@literal true} if cache keys need to be prefixed with the {@link #getKeyPrefixFor(String)} if present or
+	 * @return {@literal true} if cache keys need to be prefixed with the
+	 *         {@link #getKeyPrefixFor(String)} if present or
 	 *         the default which resolves to {@link Cache#getName()}.
 	 */
 	public boolean usePrefix() {
@@ -197,7 +208,8 @@ public Transcoder getValueTranscoder() {
 	}
 
 	/**
-	 * The name of the collection to use for this cache - if empty uses the default collection.
+	 * The name of the collection to use for this cache - if empty uses the default
+	 * collection.
 	 */
 	public String getCollectionName() {
 		return collectionName;

src/main/java/org/springframework/data/couchbase/core/AbstractTemplateSupport.java

@@ -46,8 +46,8 @@
 /**
  * Base shared by Reactive and non-Reactive TemplateSupport
  *
- * @author Emilien Bevierre
  * @author Michael Reiche
+ * @author Emilien Bevierre
  */
 @Stability.Internal
 public abstract class AbstractTemplateSupport {

src/main/java/org/springframework/data/couchbase/core/CouchbaseTemplateSupport.java

@@ -35,11 +35,11 @@
 /**
  * Internal encode/decode support for CouchbaseTemplate.
  *
- * @author Emilien Bevierre
  * @author Michael Nitschinger
  * @author Michael Reiche
  * @author Jorge Rodriguez Martin
  * @author Carlos Espinaco
+ * @author Emilien Bevierre
  * @since 3.0
  */
 class CouchbaseTemplateSupport extends AbstractTemplateSupport implements ApplicationContextAware, TemplateSupport {

src/main/java/org/springframework/data/couchbase/core/NonReactiveSupportWrapper.java

@@ -26,9 +26,9 @@
 /**
  * Wrapper of {@link TemplateSupport} methods to adapt them to {@link ReactiveTemplateSupport}.
  *
- * @author Emilien Bevierre
  * @author Carlos Espinaco
  * @author Michael Reiche
+ * @author Emilien Bevierre
  * @since 4.2
  */
 public class NonReactiveSupportWrapper implements ReactiveTemplateSupport {

src/main/java/org/springframework/data/couchbase/core/ReactiveCouchbaseTemplateSupport.java

@@ -38,9 +38,9 @@
 /**
  * Internal encode/decode support for {@link ReactiveCouchbaseTemplate}.
  *
- * @author Emilien Bevierre
  * @author Carlos Espinaco
  * @author Michael Reiche
+ * @author Emilien Bevierre
  * @since 4.2
  */
 class ReactiveCouchbaseTemplateSupport extends AbstractTemplateSupport

src/main/java/org/springframework/data/couchbase/core/ReactiveFindByIdOperationSupport.java

@@ -46,9 +46,9 @@
 /**
  * {@link ReactiveFindByIdOperation} implementations for Couchbase.
  *
- * @author Emilien Bevierre
  * @author Michael Reiche
  * @author Tigran Babloyan
+ * @author Emilien Bevierre
  */
 public class ReactiveFindByIdOperationSupport implements ReactiveFindByIdOperation {
 

src/main/java/org/springframework/data/couchbase/core/ReactiveFindFromReplicasByIdOperationSupport.java

@@ -32,8 +32,8 @@
 /**
  * {@link ReactiveFindFromReplicasByIdOperation} implementations for Couchbase.
  *
- * @author Emilien Bevierre
  * @author Michael Reiche
+ * @author Emilien Bevierre
  */
 public class ReactiveFindFromReplicasByIdOperationSupport implements ReactiveFindFromReplicasByIdOperation {
 

src/main/java/org/springframework/data/couchbase/core/ReactiveInsertByIdOperationSupport.java

@@ -43,9 +43,9 @@
 /**
  * {@link ReactiveInsertByIdOperation} implementations for Couchbase.
  *
- * @author Emilien Bevierre
  * @author Michael Reiche
  * @author Tigran Babloyan
+ * @author Emilien Bevierre
  */
 public class ReactiveInsertByIdOperationSupport implements ReactiveInsertByIdOperation {
 

src/main/java/org/springframework/data/couchbase/core/ReactiveRangeScanOperationSupport.java

@@ -31,8 +31,8 @@
 /**
  * {@link ReactiveRangeScanOperationSupport} implementations for Couchbase.
  *
- * @author Emilien Bevierre
  * @author Michael Reiche
+ * @author Emilien Bevierre
  */
 public class ReactiveRangeScanOperationSupport implements ReactiveRangeScanOperation {
 

src/main/java/org/springframework/data/couchbase/core/ReactiveRemoveByIdOperationSupport.java

@@ -44,9 +44,9 @@
 /**
  * {@link ReactiveRemoveByIdOperation} implementations for Couchbase.
  *
- * @author Emilien Bevierre
  * @author Michael Reiche
  * @author Tigran Babloyan
+ * @author Emilien Bevierre
  */
 public class ReactiveRemoveByIdOperationSupport implements ReactiveRemoveByIdOperation {