removed .equals on DeferredExecution

bbakerman · bbakerman · commit d98674b82f36 · 2025-08-01T11:00:42.000+10:00
diff --git a/src/main/java/graphql/execution/incremental/DeferredExecution.java b/src/main/java/graphql/execution/incremental/DeferredExecution.java
@@ -4,8 +4,6 @@
 import graphql.normalized.incremental.NormalizedDeferredExecution;
 import org.jspecify.annotations.Nullable;
 
-import java.util.Objects;
-
 /**
  * Represents details about the defer execution that can be associated with a {@link graphql.execution.MergedField}.
  * <p>
@@ -25,17 +23,17 @@ public String getLabel() {
         return label;
     }
 
+    // this class uses object identity - do not put .equals() / .hashCode() implementations on it
+    // otherwise it will break defer handling.  I have put the code just to be explicit that object identity
+    // is needed
+
     @Override
-    public boolean equals(Object o) {
-        if (o == null || getClass() != o.getClass()) {
-            return false;
-        }
-        DeferredExecution that = (DeferredExecution) o;
-        return Objects.equals(label, that.label);
+    public int hashCode() {
+        return super.hashCode();
     }
 
     @Override
-    public int hashCode() {
-        return Objects.hashCode(label);
+    public boolean equals(Object obj) {
+        return super.equals(obj);
     }
 }
