diff options
author | Henning Baldersheim <balder@oath.com> | 2018-11-07 00:19:23 +0100 |
---|---|---|
committer | Henning Baldersheim <balder@oath.com> | 2018-11-07 00:19:23 +0100 |
commit | 909f7a827deb71e4b7d029a4864ed3730424adcc (patch) | |
tree | d1f64927942a44a26be5c75efd0faa33f701a7db /searchcore | |
parent | 0689f93f9750f9d8c0fdbb383388b03c918ce11a (diff) |
Ensure that we do not get division by zero by always requiring non-zero divisor.
Diffstat (limited to 'searchcore')
-rw-r--r-- | searchcore/src/vespa/searchcore/fdispatch/search/fnet_search.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/searchcore/src/vespa/searchcore/fdispatch/search/fnet_search.cpp b/searchcore/src/vespa/searchcore/fdispatch/search/fnet_search.cpp index 16308ed22f9..73b92eaa2b7 100644 --- a/searchcore/src/vespa/searchcore/fdispatch/search/fnet_search.cpp +++ b/searchcore/src/vespa/searchcore/fdispatch/search/fnet_search.cpp @@ -891,7 +891,7 @@ FastS_FNET_Search::CheckCoverage() } bool missingReplies = (askedButNotAnswered != 0) || (nodesQueried != nodesReplied); const ssize_t missingParts = cntNone - (_dataset->getSearchableCopies() - 1); - if (((missingParts > 0) && (cntNone != _nodes.size())) || (missingReplies && useAdaptiveTimeout())) { + if (((missingParts > 0) || (missingReplies && useAdaptiveTimeout())) && (cntNone != _nodes.size())) { // TODO This is a dirty way of anticipating missing coverage. // It should be done differently activeDocs += missingParts * activeDocs/(_nodes.size() - cntNone); |