summaryrefslogtreecommitdiffstats
path: root/searchcore
diff options
context:
space:
mode:
authorHenning Baldersheim <balder@oath.com>2018-05-08 15:33:07 +0200
committerHenning Baldersheim <balder@oath.com>2018-05-08 15:33:07 +0200
commit1bf893499aadc36fbb8a7c30d6ad6360b5969654 (patch)
treea03eb4d8a66ebb8a19c3e0b9ed228cb1a3db73a0 /searchcore
parenta81909b9dfd416e41e4a0273b0a726619c18dcbc (diff)
Count, configured, not connected.
Diffstat (limited to 'searchcore')
-rw-r--r--searchcore/src/vespa/searchcore/fdispatch/search/fnet_search.cpp10
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);
}