diff options
author | Arne H Juul <arnej@yahooinc.com> | 2022-01-06 10:39:21 +0000 |
---|---|---|
committer | Arne H Juul <arnej@yahooinc.com> | 2022-01-06 12:02:40 +0000 |
commit | 17ba3c5161b36525b86d5522d876db101fcfc6f0 (patch) | |
tree | 88aa220fc9ac6c45dec39f36bce6fb260b121d37 /config-model/src/main/java/com/yahoo | |
parent | 4979cb8498a2cd6bb3ba7c48745a1ba7f69e2f5c (diff) |
avoid warning
Diffstat (limited to 'config-model/src/main/java/com/yahoo')
-rw-r--r-- | config-model/src/main/java/com/yahoo/vespa/model/application/validation/change/search/AttributeChangeValidator.java | 36 |
1 files changed, 18 insertions, 18 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/application/validation/change/search/AttributeChangeValidator.java b/config-model/src/main/java/com/yahoo/vespa/model/application/validation/change/search/AttributeChangeValidator.java index 1957c52e841..2a3c5fdc6b0 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/application/validation/change/search/AttributeChangeValidator.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/application/validation/change/search/AttributeChangeValidator.java @@ -109,17 +109,17 @@ public class AttributeChangeValidator { for (Attribute next : nextFields.attributes()) { Attribute current = currentFields.getAttribute(next.getName()); if (current != null) { - validateAttributeSetting(id, current, next, Attribute::isFastSearch, "fast-search", result); - validateAttributeSetting(id, current, next, Attribute::isFastAccess, "fast-access", result); - validateAttributeSetting(id, current, next, AttributeChangeValidator::extractDictionaryType, "dictionary: btree/hash", result); - validateAttributeSetting(id, current, next, AttributeChangeValidator::extractDictionaryCase, "dictionary: cased/uncased", result); - validateAttributeSetting(id, current, next, Attribute::isHuge, "huge", result); - validateAttributeSetting(id, current, next, Attribute::isPaged, "paged", result); + validateAttributePredicate(id, current, next, Attribute::isFastSearch, "fast-search", result); + validateAttributePredicate(id, current, next, Attribute::isFastAccess, "fast-access", result); + validateAttributeProperty(id, current, next, AttributeChangeValidator::extractDictionaryType, "dictionary: btree/hash", result); + validateAttributeProperty(id, current, next, AttributeChangeValidator::extractDictionaryCase, "dictionary: cased/uncased", result); + validateAttributePredicate(id, current, next, Attribute::isHuge, "huge", result); + validateAttributePredicate(id, current, next, Attribute::isPaged, "paged", result); validatePagedAttributeRemoval(current, next); - validateAttributeSetting(id, current, next, Attribute::densePostingListThreshold, "dense-posting-list-threshold", result); - validateAttributeSetting(id, current, next, Attribute::isEnabledOnlyBitVector, "rank: filter", result); - validateAttributeSetting(id, current, next, Attribute::distanceMetric, "distance-metric", result); - validateAttributeSetting(id, current, next, AttributeChangeValidator::hasHnswIndex, "indexing: index", result); + validateAttributeProperty(id, current, next, Attribute::densePostingListThreshold, "dense-posting-list-threshold", result); + validateAttributePredicate(id, current, next, Attribute::isEnabledOnlyBitVector, "rank: filter", result); + validateAttributeProperty(id, current, next, Attribute::distanceMetric, "distance-metric", result); + validateAttributePredicate(id, current, next, AttributeChangeValidator::hasHnswIndex, "indexing: index", result); if (hasHnswIndex(current) && hasHnswIndex(next)) { validateAttributeHnswIndexSetting(id, current, next, HnswIndexParams::maxLinksPerNode, "max-links-per-node", result); validateAttributeHnswIndexSetting(id, current, next, HnswIndexParams::neighborsToExploreAtInsert, "neighbors-to-explore-at-insert", result); @@ -129,10 +129,10 @@ public class AttributeChangeValidator { return result; } - private static void validateAttributeSetting(ClusterSpec.Id id, - Attribute currentAttr, Attribute nextAttr, - Predicate<Attribute> predicate, String setting, - List<VespaConfigChangeAction> result) { + private static void validateAttributePredicate(ClusterSpec.Id id, + Attribute currentAttr, Attribute nextAttr, + Predicate<Attribute> predicate, String setting, + List<VespaConfigChangeAction> result) { boolean nextValue = predicate.test(nextAttr); if (predicate.test(currentAttr) != nextValue) { String change = nextValue ? "add" : "remove"; @@ -140,10 +140,10 @@ public class AttributeChangeValidator { } } - private static <T> void validateAttributeSetting(ClusterSpec.Id id, - Attribute current, Attribute next, - Function<Attribute, T> settingValueProvider, String setting, - List<VespaConfigChangeAction> result) { + private static <T> void validateAttributeProperty(ClusterSpec.Id id, + Attribute current, Attribute next, + Function<Attribute, T> settingValueProvider, String setting, + List<VespaConfigChangeAction> result) { T currentValue = settingValueProvider.apply(current); T nextValue = settingValueProvider.apply(next); if ( ! Objects.equals(currentValue, nextValue)) { |