explanation and using Set instead of list

Signed-off-by: Attila Mészáros <[email protected]>

Author: csviri

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

@@ -1,7 +1,6 @@
 package io.javaoperatorsdk.operator.api.config;
 
 import java.time.Duration;
-import java.util.List;
 import java.util.Optional;
 import java.util.Set;
 import java.util.concurrent.ExecutorService;
@@ -464,10 +463,14 @@ default boolean previousAnnotationForDependentResourcesEventFiltering() {
    * resources. Note that if you are sure that the matcher works (most of the cases does) for your
    * case, you can remove the resource from the blocklist.
    *
+   * <p>The consequence of adding a resource type to this list is that it will not use event
+   * filtering in dependent resources, so will process also events which are results from updates of
+   * the dependent.
+   *
    * @return blocklist of resource classes where the previous version annotation won't be used.
    */
-  default List<Class<? extends HasMetadata>> previousAnnotationUsageBlocklist() {
-    return List.of(Deployment.class, StatefulSet.class);
+  default Set<Class<? extends HasMetadata>> previousAnnotationUsageBlocklist() {
+    return Set.of(Deployment.class, StatefulSet.class);
   }
 
   /**

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

@@ -1,7 +1,6 @@
 package io.javaoperatorsdk.operator.api.config;
 
 import java.time.Duration;
-import java.util.List;
 import java.util.Optional;
 import java.util.Set;
 import java.util.concurrent.ExecutorService;
@@ -41,7 +40,7 @@ public class ConfigurationServiceOverrider {
   private Boolean parseResourceVersions;
   private Boolean useSSAToPatchPrimaryResource;
   private Boolean cloneSecondaryResourcesWhenGettingFromCache;
-  private List<Class<? extends HasMetadata>> previousAnnotationUsageBlocklist;
+  private Set<Class<? extends HasMetadata>> previousAnnotationUsageBlocklist;
 
   @SuppressWarnings("rawtypes")
   private DependentResourceFactory dependentResourceFactory;
@@ -191,7 +190,7 @@ public ConfigurationServiceOverrider withCloneSecondaryResourcesWhenGettingFromC
   }
 
   public ConfigurationServiceOverrider previousAnnotationUsageBlocklist(
-      List<Class<? extends HasMetadata>> previousAnnotationUsageBlacklist) {
+      Set<Class<? extends HasMetadata>> previousAnnotationUsageBlacklist) {
     this.previousAnnotationUsageBlocklist = previousAnnotationUsageBlacklist;
     return this;
   }
@@ -338,7 +337,7 @@ public boolean cloneSecondaryResourcesWhenGettingFromCache() {
       }
 
       @Override
-      public List<Class<? extends HasMetadata>> previousAnnotationUsageBlocklist() {
+      public Set<Class<? extends HasMetadata>> previousAnnotationUsageBlocklist() {
         return overriddenValueOrDefault(
             previousAnnotationUsageBlocklist,
             ConfigurationService::previousAnnotationUsageBlocklist);