aboutsummaryrefslogtreecommitdiffstats
path: root/container-search
diff options
context:
space:
mode:
authorHenning Baldersheim <balder@yahoo-inc.com>2023-08-03 15:01:31 +0200
committerHenning Baldersheim <balder@yahoo-inc.com>2023-08-03 15:01:31 +0200
commitc1353e82644ff04a82fa0bc797f357231e9c6bd7 (patch)
tree3b25dacdca6aaf4372c8f776743a5c0cc8f914c7 /container-search
parent66c4f717422129d44aebe5e8563eacbfc0ba6fbc (diff)
Clarify error handling
Diffstat (limited to 'container-search')
-rw-r--r--container-search/src/main/java/com/yahoo/search/query/properties/QueryProperties.java2
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);