diff options
author | Jon Bratseth <bratseth@gmail.com> | 2021-12-14 18:27:52 +0100 |
---|---|---|
committer | Jon Bratseth <bratseth@gmail.com> | 2021-12-14 18:27:52 +0100 |
commit | 8a085f018f881670bb6f0907c431e37f12d77492 (patch) | |
tree | 9c155e88e4c14c1b2a187e5796449315828ba337 /documentapi | |
parent | 3a2f9ddf6b337673aeffca83b6801f9e4fa96aee (diff) |
Match document types in document selections exactly only
Routing all child types to a cluster a parent is added to
may be convenient for some users, but if it's not what you want
it is then harder to prevent it from happening.
Diffstat (limited to 'documentapi')
-rwxr-xr-x | documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/DocumentRouteSelectorPolicy.java | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/DocumentRouteSelectorPolicy.java b/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/DocumentRouteSelectorPolicy.java index 7422a303ce9..67ebe974e4e 100755 --- a/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/DocumentRouteSelectorPolicy.java +++ b/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/DocumentRouteSelectorPolicy.java @@ -39,10 +39,8 @@ public class DocumentRouteSelectorPolicy selectors.put(name, new DocumentSelector(cluster.selector())); } catch (ParseException e) { - throw new IllegalArgumentException("Error parsing selector '" + - cluster.selector() + - "' for route '" + name +"'", - e); + throw new IllegalArgumentException("Error parsing selector '" + cluster.selector() + + "' for route '" + name +"'", e); } }); this.config = Map.copyOf(selectors); @@ -87,9 +85,9 @@ public class DocumentRouteSelectorPolicy DocumentSelector selector; try { selector = new DocumentSelector(route.selector()); - log.log(Level.CONFIG, "Selector for route '" + route.name() + "' is '" + selector + "'."); + log.log(Level.CONFIG, "Selector for route '" + route.name() + "' is '" + selector + "'"); } catch (com.yahoo.document.select.parser.ParseException e) { - error = "Error parsing selector '" + route.selector() + "' for route '" + route.name() + "; " + + error = "Error parsing selector '" + route.selector() + "' for route '" + route.name() + ": " + e.getMessage(); break; } |