diff options
author | Bjørn Christian Seime <bjorncs@yahooinc.com> | 2022-12-07 18:59:25 +0100 |
---|---|---|
committer | Bjørn Christian Seime <bjorncs@yahooinc.com> | 2022-12-07 19:44:36 +0100 |
commit | 586185086d0e860a8c1e19c4dc013dbd5520f858 (patch) | |
tree | 19137d3636b25609a4509222d43894ee34f2a9f0 /vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource/DocumentV1ApiHandler.java | |
parent | fdfd6fc892245ea164729caf462aed4cac1cb6ef (diff) |
Revert back to single metric aggregated
Diffstat (limited to 'vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource/DocumentV1ApiHandler.java')
-rw-r--r-- | vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource/DocumentV1ApiHandler.java | 11 |
1 files changed, 5 insertions, 6 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 6ad6fe2b1e2..1d202ab8866 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 @@ -1100,7 +1100,7 @@ public class DocumentV1ApiHandler extends AbstractRequestHandler { incrementMetricNumOperations(); incrementMetricNumPuts(); sampleLatency(latency); switch (outcome) { case SUCCESS -> incrementMetricSucceeded(); - case CONDITION_FAILED -> { incrementMetricSucceeded(); incrementMetricConditionNotMet(); } + case CONDITION_FAILED -> incrementMetricConditionNotMet(); default -> incrementMetricFailed(); } } @@ -1110,8 +1110,8 @@ public class DocumentV1ApiHandler extends AbstractRequestHandler { incrementMetricNumOperations(); incrementMetricNumUpdates(); sampleLatency(latency); switch (outcome) { case SUCCESS -> incrementMetricSucceeded(); - case NOT_FOUND -> { incrementMetricSucceeded(); incrementMetricNotFound(); } - case CONDITION_FAILED -> { incrementMetricSucceeded(); incrementMetricConditionNotMet(); } + case NOT_FOUND -> incrementMetricNotFound(); + case CONDITION_FAILED -> incrementMetricConditionNotMet(); default -> incrementMetricFailed(); } } @@ -1119,9 +1119,8 @@ public class DocumentV1ApiHandler extends AbstractRequestHandler { private void updateRemoveMetrics(Outcome outcome, double latency) { incrementMetricNumOperations(); incrementMetricNumRemoves(); sampleLatency(latency); switch (outcome) { - case SUCCESS -> incrementMetricSucceeded(); - case NOT_FOUND -> { incrementMetricSucceeded(); incrementMetricNotFound(); } - case CONDITION_FAILED -> { incrementMetricSucceeded(); incrementMetricConditionNotMet(); } + case SUCCESS,NOT_FOUND -> incrementMetricSucceeded(); + case CONDITION_FAILED -> incrementMetricConditionNotMet(); default -> incrementMetricFailed(); } } |