diff options
author | Henning Baldersheim <balder@oath.com> | 2018-05-08 15:33:07 +0200 |
---|---|---|
committer | Henning Baldersheim <balder@oath.com> | 2018-05-08 15:33:07 +0200 |
commit | 1bf893499aadc36fbb8a7c30d6ad6360b5969654 (patch) | |
tree | a03eb4d8a66ebb8a19c3e0b9ed228cb1a3db73a0 /searchcore | |
parent | a81909b9dfd416e41e4a0273b0a726619c18dcbc (diff) |
Count, configured, not connected.
Diffstat (limited to 'searchcore')
-rw-r--r-- | searchcore/src/vespa/searchcore/fdispatch/search/fnet_search.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/searchcore/src/vespa/searchcore/fdispatch/search/fnet_search.cpp b/searchcore/src/vespa/searchcore/fdispatch/search/fnet_search.cpp index f3d5cd6ee1d..b2082826fe6 100644 --- a/searchcore/src/vespa/searchcore/fdispatch/search/fnet_search.cpp +++ b/searchcore/src/vespa/searchcore/fdispatch/search/fnet_search.cpp @@ -872,11 +872,11 @@ FastS_FNET_Search::CheckCoverage() uint16_t nodesQueried = 0; uint16_t nodesReplied = 0; size_t cntNone(0); - size_t connectedNodes(0); + size_t configuredNodes(0); for (const FastS_FNET_SearchNode & node : _nodes) { - if (node.IsConnected()) { - connectedNodes++; + if (node.GetEngine() != nullptr) { + configuredNodes++; if (node._qresult != nullptr) { covDocs += node._qresult->_coverageDocs; activeDocs += node._qresult->_activeDocs; @@ -891,10 +891,10 @@ FastS_FNET_Search::CheckCoverage() } } const ssize_t missingParts = cntNone - (_dataset->getSearchableCopies() - 1); - if ((missingParts > 0) && (cntNone != connectedNodes)) { + if ((missingParts > 0) && (cntNone != configuredNodes)) { // TODO This is a dirty way of anticipating missing coverage. // It should be done differently - activeDocs += missingParts * activeDocs/(connectedNodes - cntNone); + activeDocs += missingParts * activeDocs/(configuredNodes - cntNone); } _util.SetCoverage(covDocs, activeDocs, soonActiveDocs, degradedReason, nodesQueried, nodesReplied); } |