diff options
author | Ola Aunrønning <olaa@verizonmedia.com> | 2022-06-15 15:54:52 +0200 |
---|---|---|
committer | Ola Aunrønning <olaa@verizonmedia.com> | 2022-06-15 15:54:52 +0200 |
commit | f721aae058170c93c35ec210ea2cbd0deeab7985 (patch) | |
tree | 109f8c20f013a40bf207a2ca2555378aca15e1e4 /vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource | |
parent | 4e405a004336954ed765a3ed82a5c46792b90f78 (diff) |
conditionNotMet is not a request error
Diffstat (limited to 'vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource')
-rw-r--r-- | vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource/DocumentV1ApiHandler.java | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource/DocumentV1ApiHandler.java b/vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource/DocumentV1ApiHandler.java index af5d4c30edb..e5f9e088006 100644 --- a/vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource/DocumentV1ApiHandler.java +++ b/vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource/DocumentV1ApiHandler.java @@ -1407,11 +1407,13 @@ public class DocumentV1ApiHandler extends AbstractRequestHandler { @Override public ContentChannel handleResponse(Response response) { - switch (response.getStatus() / 100) { - case 2: metrics.reportSuccessful(type, start); break; - case 4: metrics.reportFailure(type, DocumentOperationStatus.REQUEST_ERROR); break; - case 5: metrics.reportFailure(type, DocumentOperationStatus.SERVER_ERROR); break; - } + var statusCodeGroup = response.getStatus() / 100; + if (statusCodeGroup == 2 || response.getStatus() == 412) + metrics.reportSuccessful(type, start); + else if (statusCodeGroup == 4) + metrics.reportFailure(type, DocumentOperationStatus.REQUEST_ERROR); + else if (statusCodeGroup == 5) + metrics.reportFailure(type, DocumentOperationStatus.SERVER_ERROR); return delegate.handleResponse(response); } |