aboutsummaryrefslogtreecommitdiffstats
path: root/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/ContentPolicy.java
diff options
context:
space:
mode:
Diffstat (limited to 'documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/ContentPolicy.java')
-rw-r--r--documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/ContentPolicy.java10
1 files changed, 4 insertions, 6 deletions
diff --git a/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/ContentPolicy.java b/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/ContentPolicy.java
index 03932d1b69e..5af78d97509 100644
--- a/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/ContentPolicy.java
+++ b/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/ContentPolicy.java
@@ -595,13 +595,11 @@ public class ContentPolicy extends SlobrokPolicy {
return !reply.hasErrors(); // For simplicity, count any reply with > 1 error.
}
var error = reply.getError(0);
- switch (error.getCode()) {
+ return switch (error.getCode()) {
// TODO this feels like a layering violation, but we use DocumentProtocol directly in other places in this policy anyway...
- case DocumentProtocol.ERROR_TEST_AND_SET_CONDITION_FAILED:
- case DocumentProtocol.ERROR_BUSY:
- return false;
- default: return true;
- }
+ case DocumentProtocol.ERROR_TEST_AND_SET_CONDITION_FAILED, DocumentProtocol.ERROR_BUSY -> false;
+ default -> true;
+ };
}
void handleErrorReply(Reply reply, Object untypedContext) {