From 1bf893499aadc36fbb8a7c30d6ad6360b5969654 Mon Sep 17 00:00:00 2001 From: Henning Baldersheim Date: Tue, 8 May 2018 15:33:07 +0200 Subject: Count, configured, not connected. --- .../src/vespa/searchcore/fdispatch/search/fnet_search.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'searchcore') 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); } -- cgit v1.2.3