Merge pull request #19792 from vespa-engine/bratseth/stable-query-profile-idsv7.492.78

Bratseth/stable query profile ids

1 files changed, 12 insertions, 6 deletions

diff --git a/container-search/src/main/java/com/yahoo/search/query/profile/OverridableQueryProfile.java b/container-search/src/main/java/com/yahoo/search/query/profile/OverridableQueryProfile.java
index b8de533dfbe..ac0712eeb63 100644
--- a/container-search/src/main/java/com/yahoo/search/query/profile/OverridableQueryProfile.java
+++ b/container-search/src/main/java/com/yahoo/search/query/profile/OverridableQueryProfile.java
@@ -16,12 +16,12 @@ public class OverridableQueryProfile extends QueryProfile {
     private static final String simpleClassName = OverridableQueryProfile.class.getSimpleName();
 
     /** Creates an unbacked overridable query profile */
-    protected OverridableQueryProfile() {
-        this("");
+    protected OverridableQueryProfile(QueryProfileRegistry owner) {
+        this("", owner);
     }
 
-    protected OverridableQueryProfile(String sourceName) {
-        super(ComponentId.createAnonymousComponentId(simpleClassName), sourceName);
+    protected OverridableQueryProfile(String sourceName, QueryProfileRegistry owner) {
+        super(createAnonymousId(owner), sourceName, owner);
     }
 
     @Override
@@ -35,7 +35,8 @@ public class OverridableQueryProfile extends QueryProfile {
 
     @Override
     protected QueryProfile createSubProfile(String name, DimensionBinding binding) {
-        return new OverridableQueryProfile(getSource()); // Nothing is set in this branch, so nothing to override, but need override checking
+        // Nothing is set in this branch, so nothing to override, but need override checking
+        return new OverridableQueryProfile(getSource(), getOwner());
     }
 
     /** Returns a clone of this which can be independently overridden */
@@ -43,7 +44,7 @@ public class OverridableQueryProfile extends QueryProfile {
     public OverridableQueryProfile clone() {
         if (isFrozen()) return this;
         OverridableQueryProfile clone = (OverridableQueryProfile)super.clone();
-        clone.initId(ComponentId.createAnonymousComponentId(simpleClassName));
+        clone.initId(createAnonymousId(getOwner()));
         return clone;
     }
 
@@ -52,4 +53,9 @@ public class OverridableQueryProfile extends QueryProfile {
         return "an overridable query profile with no backing";
     }
 
+    private static ComponentId createAnonymousId(QueryProfileRegistry owner) {
+        return owner != null ? owner.createAnonymousId(simpleClassName)
+                             : ComponentId.createAnonymousComponentId(simpleClassName);
+    }
+
 }