aboutsummaryrefslogtreecommitdiffstats
path: root/documentapi
diff options
context:
space:
mode:
authorJon Bratseth <bratseth@gmail.com>2021-12-14 18:27:52 +0100
committerJon Bratseth <bratseth@gmail.com>2021-12-14 18:27:52 +0100
commit8a085f018f881670bb6f0907c431e37f12d77492 (patch)
tree9c155e88e4c14c1b2a187e5796449315828ba337 /documentapi
parent3a2f9ddf6b337673aeffca83b6801f9e4fa96aee (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-xdocumentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/DocumentRouteSelectorPolicy.java10
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;
}