refactor: introduce IndexedResourceCache interface

Signed-off-by: Chris Laprun <claprun@redhat.com>

Author: metacosm

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/Context.java

@@ -10,6 +10,7 @@
 import io.javaoperatorsdk.operator.api.config.ControllerConfiguration;
 import io.javaoperatorsdk.operator.api.reconciler.dependent.managed.ManagedDependentResourceContext;
 import io.javaoperatorsdk.operator.processing.event.EventSourceRetriever;
+import io.javaoperatorsdk.operator.processing.event.source.IndexedResourceCache;
 import io.javaoperatorsdk.operator.processing.event.source.IndexerResourceCache;
 
 public interface Context<P extends HasMetadata> {
@@ -51,5 +52,5 @@ <R> Optional<R> getSecondaryResource(Class<R> expectedType,
    * @return the {@link IndexerResourceCache} associated with the associated {@link Reconciler} for
    *         this context
    */
-  IndexerResourceCache<P> getPrimaryCache();
+  IndexedResourceCache<P> getPrimaryCache();
 }

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/DefaultContext.java

@@ -13,7 +13,7 @@
 import io.javaoperatorsdk.operator.api.reconciler.dependent.managed.ManagedDependentResourceContext;
 import io.javaoperatorsdk.operator.processing.Controller;
 import io.javaoperatorsdk.operator.processing.event.EventSourceRetriever;
-import io.javaoperatorsdk.operator.processing.event.source.IndexerResourceCache;
+import io.javaoperatorsdk.operator.processing.event.source.IndexedResourceCache;
 
 public class DefaultContext<P extends HasMetadata> implements Context<P> {
 
@@ -42,7 +42,7 @@ public <T> Set<T> getSecondaryResources(Class<T> expectedType) {
   }
 
   @Override
-  public IndexerResourceCache<P> getPrimaryCache() {
+  public IndexedResourceCache<P> getPrimaryCache() {
     return controller.getEventSourceManager().getControllerResourceEventSource();
   }
 

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/IndexedResourceCache.java

@@ -0,0 +1,9 @@
+package io.javaoperatorsdk.operator.processing.event.source;
+
+import io.fabric8.kubernetes.api.model.HasMetadata;
+
+import java.util.List;
+
+public interface IndexedResourceCache<T extends HasMetadata> extends ResourceCache<T> {
+    List<T> byIndex(String indexName, String indexKey);
+}

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/IndexerResourceCache.java

@@ -6,14 +6,11 @@
 
 import io.fabric8.kubernetes.api.model.HasMetadata;
 
-public interface IndexerResourceCache<T extends HasMetadata> extends ResourceCache<T> {
+public interface IndexerResourceCache<T extends HasMetadata> extends IndexedResourceCache<T> {
 
   void addIndexers(Map<String, Function<T, List<String>>> indexers);
 
   default void addIndexer(String name, Function<T, List<String>> indexer) {
     addIndexers(Map.of(name, indexer));
   }
-
-  List<T> byIndex(String indexName, String indexKey);
-
 }