diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2018-03-15 19:20:00 +0100 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2018-03-15 19:20:00 +0100 |
commit | 1d7707af2976152055d7e05621f8af0aa4266d2c (patch) | |
tree | cd19bc22dd616dc2147222931bb7f45425b804d6 /container-search/src/main/java/com/yahoo/prelude/statistics/StatisticsSearcher.java | |
parent | 9dce3a96124a8a0a2c1ca10cdc941f582a44a4e1 (diff) |
Ensure we handle null coverage object too.
Diffstat (limited to 'container-search/src/main/java/com/yahoo/prelude/statistics/StatisticsSearcher.java')
-rw-r--r-- | container-search/src/main/java/com/yahoo/prelude/statistics/StatisticsSearcher.java | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/container-search/src/main/java/com/yahoo/prelude/statistics/StatisticsSearcher.java b/container-search/src/main/java/com/yahoo/prelude/statistics/StatisticsSearcher.java index a231dc8f612..899acabe15c 100644 --- a/container-search/src/main/java/com/yahoo/prelude/statistics/StatisticsSearcher.java +++ b/container-search/src/main/java/com/yahoo/prelude/statistics/StatisticsSearcher.java @@ -240,13 +240,15 @@ public class StatisticsSearcher extends Searcher { incrementStatePageOnlyErrors(result); } Coverage queryCoverage = result.getCoverage(false); - if (queryCoverage != null && queryCoverage.isDegraded()) { - Metric.Context degradedContext = getDegradedMetricContext(execution.chain().getId().stringValue(), queryCoverage); - metric.add(DEGRADED_METRIC, 1, degradedContext); + if (queryCoverage != null) { + if (queryCoverage.isDegraded()) { + Metric.Context degradedContext = getDegradedMetricContext(execution.chain().getId().stringValue(), queryCoverage); + metric.add(DEGRADED_METRIC, 1, degradedContext); + } + metric.set(COVERAGE_METRIC, (double) queryCoverage.getResultPercentage(), metricContext); } int hitCount = result.getConcreteHitCount(); hitsPerQuery.put((double) hitCount); - metric.set(COVERAGE_METRIC, (double) queryCoverage.getResultPercentage(), metricContext); metric.set(HITS_PER_QUERY_METRIC, (double) hitCount, metricContext); metric.set(TOTALHITS_PER_QUERY_METRIC, (double) result.getTotalHitCount(), metricContext); if (hitCount == 0) { |