diff options
author | Bjørn Christian Seime <bjorncs@yahooinc.com> | 2023-01-09 14:20:36 +0100 |
---|---|---|
committer | Bjørn Christian Seime <bjorncs@yahooinc.com> | 2023-01-09 14:20:36 +0100 |
commit | 74422f710be52c00f7edd86e24d5e054c93c24e8 (patch) | |
tree | 2ae867929747d36d0e5039fcc0e6ee01967701f1 /vespaclient-container-plugin/src/main/java | |
parent | b22d7448f85fd28818eafde6794840e85b05d7cc (diff) |
Don't classify condition-not-met and not-found as success
Diffstat (limited to 'vespaclient-container-plugin/src/main/java')
-rw-r--r-- | vespaclient-container-plugin/src/main/java/com/yahoo/vespa/http/server/FeedReplyReader.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/vespaclient-container-plugin/src/main/java/com/yahoo/vespa/http/server/FeedReplyReader.java b/vespaclient-container-plugin/src/main/java/com/yahoo/vespa/http/server/FeedReplyReader.java index 1422ec10b08..377e91f6490 100644 --- a/vespaclient-container-plugin/src/main/java/com/yahoo/vespa/http/server/FeedReplyReader.java +++ b/vespaclient-container-plugin/src/main/java/com/yahoo/vespa/http/server/FeedReplyReader.java @@ -53,11 +53,13 @@ public class FeedReplyReader implements ReplyHandler { enqueue(context, reply.getError(0).getMessage(), ErrorCode.ERROR, false, reply.getTrace()); } else { metricsHelper.reportSuccessful(type, latencyInSeconds); - metric.add(MetricNames.SUCCEEDED, 1, null); if ( ! conditionMet) metric.add(MetricNames.CONDITION_NOT_MET, 1, testAndSetMetricCtx); - if ( ! updateNotFound(reply)) + else if ( ! updateNotFound(reply)) metric.add(MetricNames.NOT_FOUND, 1, null); + else + metric.add(MetricNames.SUCCEEDED, 1, null); + enqueue(context, "Document processed.", ErrorCode.OK, !conditionMet, reply.getTrace()); } } |