diff options
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.java | 10 |
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) { |