summaryrefslogtreecommitdiffstats
path: root/searchlib
diff options
context:
space:
mode:
authorArne Juul <arnej@verizonmedia.com>2020-06-17 14:24:24 +0000
committerArne Juul <arnej@verizonmedia.com>2020-06-17 14:34:25 +0000
commit43d9d52c85ca09bdacc5b35bf69b3c0f0adb277f (patch)
tree7bed82730efd333ceb762947a978822b34b2fcf7 /searchlib
parent44ebaa3f2345a17f2d90f1fd5cbb3463e4fd9b1f (diff)
rename me -> self_docid
* other tests should now ensure that we don't get self-references.
Diffstat (limited to 'searchlib')
-rw-r--r--searchlib/src/vespa/searchlib/tensor/hnsw_index.cpp5
1 files changed, 3 insertions, 2 deletions
diff --git a/searchlib/src/vespa/searchlib/tensor/hnsw_index.cpp b/searchlib/src/vespa/searchlib/tensor/hnsw_index.cpp
index c9474eaf483..36d970dfd01 100644
--- a/searchlib/src/vespa/searchlib/tensor/hnsw_index.cpp
+++ b/searchlib/src/vespa/searchlib/tensor/hnsw_index.cpp
@@ -335,7 +335,7 @@ HnswIndex::internal_prepare_add(uint32_t docid, TypedCells input_vector) const
}
HnswIndex::LinkArray
-HnswIndex::filter_valid_docids(uint32_t level, const PreparedAddDoc::Links &neighbors, uint32_t me)
+HnswIndex::filter_valid_docids(uint32_t level, const PreparedAddDoc::Links &neighbors, uint32_t self_docid)
{
LinkArray valid;
valid.reserve(neighbors.size());
@@ -343,8 +343,9 @@ HnswIndex::filter_valid_docids(uint32_t level, const PreparedAddDoc::Links &neig
uint32_t docid = neighbor.first;
HnswGraph::NodeRef node_ref = neighbor.second;
if (_graph.still_valid(docid, node_ref)) {
+ assert(docid != self_docid);
auto levels = _graph.get_level_array(node_ref);
- if ((docid != me) && (level < levels.size())) {
+ if (level < levels.size()) {
valid.push_back(docid);
}
}