diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2023-08-03 15:01:31 +0200 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2023-08-03 15:01:31 +0200 |
commit | c1353e82644ff04a82fa0bc797f357231e9c6bd7 (patch) | |
tree | 3b25dacdca6aaf4372c8f776743a5c0cc8f914c7 /container-search/src/main/java/com/yahoo/search/query | |
parent | 66c4f717422129d44aebe5e8563eacbfc0ba6fbc (diff) |
Clarify error handling
Diffstat (limited to 'container-search/src/main/java/com/yahoo/search/query')
-rw-r--r-- | container-search/src/main/java/com/yahoo/search/query/properties/QueryProperties.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/container-search/src/main/java/com/yahoo/search/query/properties/QueryProperties.java b/container-search/src/main/java/com/yahoo/search/query/properties/QueryProperties.java index 896a916aa50..800b3a1ba89 100644 --- a/container-search/src/main/java/com/yahoo/search/query/properties/QueryProperties.java +++ b/container-search/src/main/java/com/yahoo/search/query/properties/QueryProperties.java @@ -169,7 +169,6 @@ public class QueryProperties extends Properties { return; } - //TODO Why is there error handling in set path and not in get path ? if (key.first().equals(Ranking.RANKING)) { if (key.size() > 2) { String restKey = key.rest().rest().toString(); @@ -189,6 +188,7 @@ public class QueryProperties extends Properties { } } if (reservedPrefix.contains(key.first())) { + // Setting a property under the reserved paths are illegal, while retrieving(get) one is not. throwIllegalParameter(key.rest().toString(), key.first()); } else { super.set(key, value, context); |