diff options
author | Arne Juul <arnej@verizonmedia.com> | 2020-06-17 14:24:24 +0000 |
---|---|---|
committer | Arne Juul <arnej@verizonmedia.com> | 2020-06-17 14:34:25 +0000 |
commit | 43d9d52c85ca09bdacc5b35bf69b3c0f0adb277f (patch) | |
tree | 7bed82730efd333ceb762947a978822b34b2fcf7 /searchlib | |
parent | 44ebaa3f2345a17f2d90f1fd5cbb3463e4fd9b1f (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.cpp | 5 |
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); } } |