diff options
author | Jon Bratseth <bratseth@gmail.com> | 2022-03-31 10:00:17 +0200 |
---|---|---|
committer | Jon Bratseth <bratseth@gmail.com> | 2022-03-31 10:00:17 +0200 |
commit | bc957f4d0196c043485b157c293b1163bc963b68 (patch) | |
tree | 1fff6893f12be15bcd845ac2233a2ef7350a339c /container-search/src/main/java/com/yahoo/search/Query.java | |
parent | 0ce79641b9c8bb4cb12ecd1a61b052e4c4742efe (diff) |
Slightly more validation
Diffstat (limited to 'container-search/src/main/java/com/yahoo/search/Query.java')
-rw-r--r-- | container-search/src/main/java/com/yahoo/search/Query.java | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/container-search/src/main/java/com/yahoo/search/Query.java b/container-search/src/main/java/com/yahoo/search/Query.java index 3d411adeb75..7df243404a5 100644 --- a/container-search/src/main/java/com/yahoo/search/Query.java +++ b/container-search/src/main/java/com/yahoo/search/Query.java @@ -367,11 +367,10 @@ public class Query extends com.yahoo.processing.Request implements Cloneable { ZoneInfo zoneInfo) { startTime = httpRequest.getJDiscRequest().creationTime(TimeUnit.MILLISECONDS); if (queryProfile != null) { - // Move all request parameters to the query profile just to validate that the parameter settings are legal + // Move all request parameters to the query profile Properties queryProfileProperties = new QueryProfileProperties(queryProfile, embedders); properties().chain(queryProfileProperties); - // TODO: Just checking legality rather than actually setting would be faster - setPropertiesFromRequestMap(requestMap, properties(), true); // Adds errors to the query for illegal set attempts + setPropertiesFromRequestMap(requestMap, properties(), true); // Create the full chain properties().chain(new QueryProperties(this, queryProfile.getRegistry(), embedders)). |