diff options
author | Jon Bratseth <bratseth@gmail.com> | 2020-07-03 13:43:27 +0200 |
---|---|---|
committer | Jon Bratseth <bratseth@gmail.com> | 2020-07-03 13:43:27 +0200 |
commit | 25367bc45e5accc2d93ad9559e740ca317c0cf4a (patch) | |
tree | 821335b66c4dfb34d9f43d6fd53ad85fe2465c72 /container-search/src/main/java/com/yahoo/search/query/profile | |
parent | 566d9a7d5cf2df8e3a0746522c5f0936977a3b8f (diff) |
Value nodes have priority over non-value nodes
Diffstat (limited to 'container-search/src/main/java/com/yahoo/search/query/profile')
-rw-r--r-- | container-search/src/main/java/com/yahoo/search/query/profile/AllValuesQueryProfileVisitor.java | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/container-search/src/main/java/com/yahoo/search/query/profile/AllValuesQueryProfileVisitor.java b/container-search/src/main/java/com/yahoo/search/query/profile/AllValuesQueryProfileVisitor.java index 2b61dc4c0a6..3c336c80d37 100644 --- a/container-search/src/main/java/com/yahoo/search/query/profile/AllValuesQueryProfileVisitor.java +++ b/container-search/src/main/java/com/yahoo/search/query/profile/AllValuesQueryProfileVisitor.java @@ -47,7 +47,11 @@ final class AllValuesQueryProfileVisitor extends PrefixQueryProfileVisitor { DimensionValues variant, DimensionBinding binding) { CompoundName fullName = currentPrefix.append(key); - if (values.containsKey(fullName.toString())) return; // The first value encountered has priority + + ValueWithSource existing = values.get(fullName.toString()); + + // The first value encountered has priority and values have priority over profiles + if (existing != null && (existing.value() != null || value == null)) return; Boolean isOverridable = owner != null ? owner.isLocalOverridable(key, binding) : null; |