improve: deprecate instance based Mappers.fromOwnerReferences

csviri · csviri · commit b3d8f323958f · 2026-05-19T22:14:46.000+02:00
This does not make sense to me, this should be always based on class

Signed-off-by: Attila Mészáros &lt;a_meszaros@apple.com&gt;
diff --git a/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/informer/Mappers.java b/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/informer/Mappers.java
@@ -87,11 +87,13 @@ public static <T extends HasMetadata> SecondaryToPrimaryMapper<T> fromOwnerRefer
         clusterScoped);
   }
 
+  @Deprecated(forRemoval = true)
   public static <T extends HasMetadata> SecondaryToPrimaryMapper<T> fromOwnerReferences(
       HasMetadata primaryResource) {
     return fromOwnerReferences(primaryResource, false);
   }
 
+  @Deprecated(forRemoval = true)
   public static <T extends HasMetadata> SecondaryToPrimaryMapper<T> fromOwnerReferences(
       HasMetadata primaryResource, boolean clusterScoped) {
     return fromOwnerReferences(

Original file line number	Diff line number	Diff line change
`@@ -87,11 +87,13 @@ public static <T extends HasMetadata> SecondaryToPrimaryMapper<T> fromOwnerRefer`
`87`	`87`	`clusterScoped);`
`88`	`88`	`}`
`89`	`89`
	`90`	`+ @Deprecated(forRemoval = true)`
`90`	`91`	`public static <T extends HasMetadata> SecondaryToPrimaryMapper<T> fromOwnerReferences(`
`91`	`92`	`HasMetadata primaryResource) {`
`92`	`93`	`return fromOwnerReferences(primaryResource, false);`
`93`	`94`	`}`
`94`	`95`
	`96`	`+ @Deprecated(forRemoval = true)`
`95`	`97`	`public static <T extends HasMetadata> SecondaryToPrimaryMapper<T> fromOwnerReferences(`
`96`	`98`	`HasMetadata primaryResource, boolean clusterScoped) {`
`97`	`99`	`return fromOwnerReferences(`