Fix lint error

Signed-off-by: 10000-ki <[email protected]>

Author: 10000-ki

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/ConfigurationService.java

@@ -56,10 +56,10 @@ public interface ConfigurationService {
    * </p>
    *
    * @param baseConfiguration the {@link ConfigurationService} to start from
-   * @param overrider the {@link ConfigurationServiceOverrider} used to change the values provided
-   *        by the base configuration
+   * @param overrider         the {@link ConfigurationServiceOverrider} used to change the values
+   *                          provided by the base configuration
    * @return a new {@link ConfigurationService} starting from the configuration provided as base but
-   *         with overridden values.
+   * with overridden values.
    */
   static ConfigurationService newOverriddenConfigurationService(
       ConfigurationService baseConfiguration,
@@ -86,9 +86,9 @@ static ConfigurationService newOverriddenConfigurationService(
    * </p>
    *
    * @param overrider the {@link ConfigurationServiceOverrider} used to change the values provided
-   *        by the default configuration
+   *                  by the default configuration
    * @return a new {@link ConfigurationService} overriding the default values with the ones provided
-   *         by the specified {@link ConfigurationServiceOverrider}
+   * by the specified {@link ConfigurationServiceOverrider}
    * @since 4.4.0
    */
   static ConfigurationService newOverriddenConfigurationService(
@@ -100,9 +100,9 @@ static ConfigurationService newOverriddenConfigurationService(
    * Retrieves the configuration associated with the specified reconciler
    *
    * @param reconciler the reconciler we want the configuration of
-   * @param <R> the {@code CustomResource} type associated with the specified reconciler
-   * @return the {@link ControllerConfiguration} associated with the specified reconciler or {@code
-   * null} if no configuration exists for the reconciler
+   * @param <R>        the {@code CustomResource} type associated with the specified reconciler
+   * @return the {@link ControllerConfiguration} associated with the specified reconciler or
+   * {@code null} if no configuration exists for the reconciler
    */
   <R extends HasMetadata> ControllerConfiguration<R> getConfigurationFor(Reconciler<R> reconciler);
 
@@ -223,7 +223,7 @@ default Metrics getMetrics() {
    * handle concurrent reconciliations
    *
    * @return the {@link ExecutorService} implementation to use for concurrent reconciliation
-   *         processing
+   * processing
    */
   default ExecutorService getExecutorService() {
     return Executors.newFixedThreadPool(concurrentReconciliationThreads());
@@ -251,7 +251,8 @@ default boolean closeClientOnStop() {
 
   /**
    * Override to provide a custom {@link DependentResourceFactory} implementation to change how
-   * {@link io.javaoperatorsdk.operator.api.reconciler.dependent.DependentResource} are instantiated
+   * {@link io.javaoperatorsdk.operator.api.reconciler.dependent.DependentResource} are
+   * instantiated
    *
    * @return the custom {@link DependentResourceFactory} implementation
    */
@@ -300,8 +301,8 @@ default Duration cacheSyncTimeout() {
   }
 
   /**
-   * This is the timeout value that allows the reconciliation threads to gracefully shut down.
-   * If no value is set, the default is immediate shutdown.
+   * This is the timeout value that allows the reconciliation threads to gracefully shut down. If no
+   * value is set, the default is immediate shutdown.
    *
    * @return The duration of time to wait before terminating the reconciliation threads
    */
@@ -363,9 +364,8 @@ default ExecutorServiceManager getExecutorServiceManager() {
    * SSA based create/update can be still used with the legacy matching, just overriding the match
    * method of Kubernetes Dependent Resource.
    *
-   * @since 4.4.0
-   *
    * @return if SSA should be used for dependent resources
+   * @since 4.4.0
    */
   default boolean ssaBasedCreateUpdateMatchForDependentResources() {
     return true;
@@ -393,9 +393,8 @@ default Set<Class<? extends HasMetadata>> defaultNonSSAResource() {
    * <p>
    * Disable this if you want to react to your own dependent resource updates
    *
-   * @since 4.5.0
-   *
    * @return if special annotation should be used for dependent resource to filter events
+   * @since 4.5.0
    */
   default boolean previousAnnotationForDependentResourcesEventFiltering() {
     return true;
@@ -410,9 +409,8 @@ default boolean previousAnnotationForDependentResourcesEventFiltering() {
    * logic, and you want to further minimize the amount of work done / updates issued by the
    * operator.
    *
-   * @since 4.5.0
-   *
    * @return if resource version should be parsed (as integer)
+   * @since 4.5.0
    */
   default boolean parseResourceVersionsForEventFilteringAndCaching() {
     return false;
@@ -424,9 +422,9 @@ default boolean parseResourceVersionsForEventFilteringAndCaching() {
    * adding finalizers, patching resources and status.
    *
    * @return {@code true} if Server-Side Apply (SSA) should be used when patching the primary
-   *         resources, {@code false} otherwise
-   * @since 5.0.0
+   * resources, {@code false} otherwise
    * @see ConfigurationServiceOverrider#withUseSSAToPatchPrimaryResource(boolean)
+   * @since 5.0.0
    */
   default boolean useSSAToPatchPrimaryResource() {
     return true;
@@ -447,8 +445,7 @@ default boolean useSSAToPatchPrimaryResource() {
    * </p>
    *
    * @return {@code true} if resources should be defensively cloned before returning them from
-   *         caches, {@code false} otherwise
-   *
+   * caches, {@code false} otherwise
    * @since 5.0.0
    */
   default boolean cloneSecondaryResourcesWhenGettingFromCache() {

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/ConfigurationServiceOverrider.java

@@ -14,7 +14,7 @@
 import io.javaoperatorsdk.operator.api.monitoring.Metrics;
 import io.javaoperatorsdk.operator.api.reconciler.dependent.DependentResourceFactory;
 
-@SuppressWarnings({"unused", "UnusedReturnValue"})
+@SuppressWarnings( {"unused", "UnusedReturnValue"})
 public class ConfigurationServiceOverrider {
 
   private static final Logger log = LoggerFactory.getLogger(ConfigurationServiceOverrider.class);

operator-framework-core/src/test/java/io/javaoperatorsdk/operator/api/config/ConfigurationServiceOverriderTest.java

@@ -13,7 +13,8 @@
 
 class ConfigurationServiceOverriderTest {
 
-  private static final Metrics METRICS = new Metrics() {};
+  private static final Metrics METRICS = new Metrics() {
+  };
 
   private static final LeaderElectionConfiguration LEADER_ELECTION_CONFIGURATION =
       new LeaderElectionConfiguration("foo", "fooNS");
@@ -60,7 +61,8 @@ public <R extends HasMetadata> R clone(R object) {
           }
         })
         .withConcurrentReconciliationThreads(25)
-        .withMetrics(new Metrics() {})
+        .withMetrics(new Metrics() {
+        })
         .withLeaderElectionConfiguration(new LeaderElectionConfiguration("newLease", "newLeaseNS"))
         .withInformerStoppedHandler((informer, ex) -> {
         })
@@ -79,7 +81,8 @@ public <R extends HasMetadata> R clone(R object) {
         overridden.getLeaderElectionConfiguration());
     assertNotEquals(config.getInformerStoppedHandler(),
         overridden.getLeaderElectionConfiguration());
-    assertNotEquals(config.reconciliationTerminationTimeout(), overridden.reconciliationTerminationTimeout());
+    assertNotEquals(config.reconciliationTerminationTimeout(),
+        overridden.reconciliationTerminationTimeout());
   }
 
 }