diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2024-01-16 20:05:25 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-01-16 20:05:25 +0100 |
commit | df5d011fffe330efc8321b0f0050cae03d45cf91 (patch) | |
tree | 5a7e73a95bf2f715faa1cb2f64246f1663153c4d | |
parent | ada510e89058979aecd66598d1853fd09ee0b42d (diff) | |
parent | 1c5d19fcb4ee457737e70c277f7ceb946c5f174e (diff) |
Merge pull request #29935 from vespa-engine/geirst/multi-term-hash-filter
Rename MultiTermFilter to MultiTermHashFilter.
-rw-r--r-- | searchlib/src/vespa/searchlib/attribute/attribute_weighted_set_blueprint.cpp | 4 | ||||
-rw-r--r-- | searchlib/src/vespa/searchlib/attribute/multi_term_hash_filter.h (renamed from searchlib/src/vespa/searchlib/attribute/multi_term_filter.h) | 8 | ||||
-rw-r--r-- | searchlib/src/vespa/searchlib/attribute/multi_term_hash_filter.hpp (renamed from searchlib/src/vespa/searchlib/attribute/multi_term_filter.hpp) | 14 |
3 files changed, 13 insertions, 13 deletions
diff --git a/searchlib/src/vespa/searchlib/attribute/attribute_weighted_set_blueprint.cpp b/searchlib/src/vespa/searchlib/attribute/attribute_weighted_set_blueprint.cpp index 99d3ba3f7aa..01148c11c9c 100644 --- a/searchlib/src/vespa/searchlib/attribute/attribute_weighted_set_blueprint.cpp +++ b/searchlib/src/vespa/searchlib/attribute/attribute_weighted_set_blueprint.cpp @@ -1,7 +1,7 @@ // Copyright Vespa.ai. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. #include "attribute_weighted_set_blueprint.h" -#include "multi_term_filter.hpp" +#include "multi_term_hash_filter.hpp" #include <vespa/searchcommon/attribute/i_search_context.h> #include <vespa/searchlib/common/bitvector.h> #include <vespa/searchlib/fef/matchdatalayout.h> @@ -73,7 +73,7 @@ make_multi_term_filter(fef::TermFieldMatchData& tfmd, const std::vector<int32_t>& weights, const std::vector<ISearchContext*>& contexts) { - using FilterType = attribute::MultiTermFilter<WrapperType>; + using FilterType = attribute::MultiTermHashFilter<WrapperType>; typename FilterType::TokenMap tokens; WrapperType wrapper(attr); for (size_t i = 0; i < contexts.size(); ++i) { diff --git a/searchlib/src/vespa/searchlib/attribute/multi_term_filter.h b/searchlib/src/vespa/searchlib/attribute/multi_term_hash_filter.h index adbf37d2dcd..9c3ea258fdc 100644 --- a/searchlib/src/vespa/searchlib/attribute/multi_term_filter.h +++ b/searchlib/src/vespa/searchlib/attribute/multi_term_hash_filter.h @@ -18,7 +18,7 @@ namespace search::attribute { * @tparam WrapperType Type that wraps an attribute vector and provides access to the attribute value for a given docid. */ template <typename WrapperType> -class MultiTermFilter final : public queryeval::SearchIterator +class MultiTermHashFilter final : public queryeval::SearchIterator { public: using Key = typename WrapperType::TokenT; @@ -31,9 +31,9 @@ private: int32_t _weight; public: - MultiTermFilter(fef::TermFieldMatchData& tfmd, - WrapperType attr, - TokenMap&& map); + MultiTermHashFilter(fef::TermFieldMatchData& tfmd, + WrapperType attr, + TokenMap&& map); void and_hits_into(BitVector& result, uint32_t begin_id) override; void doSeek(uint32_t docId) override; diff --git a/searchlib/src/vespa/searchlib/attribute/multi_term_filter.hpp b/searchlib/src/vespa/searchlib/attribute/multi_term_hash_filter.hpp index dc572aedbff..96d5b3ac1f3 100644 --- a/searchlib/src/vespa/searchlib/attribute/multi_term_filter.hpp +++ b/searchlib/src/vespa/searchlib/attribute/multi_term_hash_filter.hpp @@ -2,16 +2,16 @@ #pragma once -#include "multi_term_filter.h" +#include "multi_term_hash_filter.h" #include <vespa/searchlib/common/bitvector.h> #include <vespa/searchlib/fef/termfieldmatchdata.h> namespace search::attribute { template <typename WrapperType> -MultiTermFilter<WrapperType>::MultiTermFilter(fef::TermFieldMatchData& tfmd, - WrapperType attr, - TokenMap&& map) +MultiTermHashFilter<WrapperType>::MultiTermHashFilter(fef::TermFieldMatchData& tfmd, + WrapperType attr, + TokenMap&& map) : _tfmd(tfmd), _attr(attr), _map(std::move(map)), @@ -21,7 +21,7 @@ MultiTermFilter<WrapperType>::MultiTermFilter(fef::TermFieldMatchData& tfmd, template <typename WrapperType> void -MultiTermFilter<WrapperType>::and_hits_into(BitVector& result, uint32_t begin_id) +MultiTermHashFilter<WrapperType>::and_hits_into(BitVector& result, uint32_t begin_id) { auto end = _map.end(); result.foreach_truebit([&, end](uint32_t key) { if ( _map.find(_attr.getToken(key)) == end) { result.clearBit(key); }}, begin_id); @@ -29,7 +29,7 @@ MultiTermFilter<WrapperType>::and_hits_into(BitVector& result, uint32_t begin_id template <typename WrapperType> void -MultiTermFilter<WrapperType>::doSeek(uint32_t docId) +MultiTermHashFilter<WrapperType>::doSeek(uint32_t docId) { auto pos = _map.find(_attr.getToken(docId)); if (pos != _map.end()) { @@ -40,7 +40,7 @@ MultiTermFilter<WrapperType>::doSeek(uint32_t docId) template <typename WrapperType> void -MultiTermFilter<WrapperType>::doUnpack(uint32_t docId) +MultiTermHashFilter<WrapperType>::doUnpack(uint32_t docId) { _tfmd.reset(docId); fef::TermFieldMatchDataPosition pos; |