diff options
Diffstat (limited to 'streamingvisitors/src/vespa/vsm/searcher')
6 files changed, 41 insertions, 26 deletions
diff --git a/streamingvisitors/src/vespa/vsm/searcher/tokenizereader.cpp b/streamingvisitors/src/vespa/vsm/searcher/tokenizereader.cpp index d8a6091fe11..5988bdd912f 100644 --- a/streamingvisitors/src/vespa/vsm/searcher/tokenizereader.cpp +++ b/streamingvisitors/src/vespa/vsm/searcher/tokenizereader.cpp @@ -4,6 +4,19 @@ namespace vsm { +namespace { + +template <bool exact_match> inline bool is_word_char(ucs4_t c); + +template <> +inline bool is_word_char<false>(ucs4_t c) { return Fast_UnicodeUtil::IsWordChar(c); } + +// All characters are treated as word characters for exact match +template <> +inline constexpr bool is_word_char<true>(ucs4_t) { return true; } + +} + void TokenizeReader::fold(ucs4_t c) { const char *repl = Fast_NormalizeWordFolder::ReplacementString(c); @@ -18,4 +31,24 @@ TokenizeReader::fold(ucs4_t c) { } } +template <bool exact_match> +size_t +TokenizeReader::tokenize_helper(Normalizing norm_mode) +{ + ucs4_t c(0); + while (hasNext()) { + if (is_word_char<exact_match>(c = next())) { + normalize(c, norm_mode); + while (hasNext() && is_word_char<exact_match>(c = next())) { + normalize(c, norm_mode); + } + break; + } + } + return complete(); +} + +template size_t TokenizeReader::tokenize_helper<false>(Normalizing); +template size_t TokenizeReader::tokenize_helper<true>(Normalizing); + } diff --git a/streamingvisitors/src/vespa/vsm/searcher/tokenizereader.h b/streamingvisitors/src/vespa/vsm/searcher/tokenizereader.h index 2bb5e62e0aa..f680d9b6c47 100644 --- a/streamingvisitors/src/vespa/vsm/searcher/tokenizereader.h +++ b/streamingvisitors/src/vespa/vsm/searcher/tokenizereader.h @@ -43,6 +43,10 @@ public: _q = _q_start; return token_len; } + template <bool exact_match> + size_t tokenize_helper(Normalizing norm_mode); + size_t tokenize(Normalizing norm_mode) { return tokenize_helper<false>(norm_mode); } + size_t tokenize_exact_match(Normalizing norm_mode) { return tokenize_helper<true>(norm_mode); } private: void fold(ucs4_t c); const byte *_p; diff --git a/streamingvisitors/src/vespa/vsm/searcher/utf8strchrfieldsearcher.cpp b/streamingvisitors/src/vespa/vsm/searcher/utf8strchrfieldsearcher.cpp index 37dc4ffb99c..c860178d583 100644 --- a/streamingvisitors/src/vespa/vsm/searcher/utf8strchrfieldsearcher.cpp +++ b/streamingvisitors/src/vespa/vsm/searcher/utf8strchrfieldsearcher.cpp @@ -26,8 +26,7 @@ UTF8StrChrFieldSearcher::matchTerms(const FieldRef & f, size_t mintsz) TokenizeReader reader(reinterpret_cast<const byte *> (f.data()), f.size(), fn); while ( reader.hasNext() ) { - tokenize(reader); - size_t fl = reader.complete(); + size_t fl = reader.tokenize(normalize_mode()); for (auto qt : _qtl) { const cmptype_t * term; termsize_t tsz = qt->term(term); diff --git a/streamingvisitors/src/vespa/vsm/searcher/utf8stringfieldsearcherbase.cpp b/streamingvisitors/src/vespa/vsm/searcher/utf8stringfieldsearcherbase.cpp index 5036e9bedb1..f016d08ece8 100644 --- a/streamingvisitors/src/vespa/vsm/searcher/utf8stringfieldsearcherbase.cpp +++ b/streamingvisitors/src/vespa/vsm/searcher/utf8stringfieldsearcherbase.cpp @@ -10,21 +10,6 @@ using search::byte; namespace vsm { -template<typename Reader> -void -UTF8StringFieldSearcherBase::tokenize(Reader & reader) { - ucs4_t c(0); - Normalizing norm_mode = normalize_mode(); - while (reader.hasNext() && ! Fast_UnicodeUtil::IsWordChar(c = reader.next())); - - if (Fast_UnicodeUtil::IsWordChar(c)) { - reader.normalize(c, norm_mode); - while (reader.hasNext() && Fast_UnicodeUtil::IsWordChar(c = reader.next())) { - reader.normalize(c, norm_mode); - } - } -} - size_t UTF8StringFieldSearcherBase::matchTermRegular(const FieldRef & f, QueryTerm & qt) { @@ -38,8 +23,7 @@ UTF8StringFieldSearcherBase::matchTermRegular(const FieldRef & f, QueryTerm & qt TokenizeReader reader(reinterpret_cast<const byte *> (f.data()), f.size(), fn); while ( reader.hasNext() ) { - tokenize(reader); - size_t fl = reader.complete(); + size_t fl = reader.tokenize(normalize_mode()); if ((tsz <= fl) && (prefix() || qt.isPrefix() || (tsz == fl))) { const cmptype_t *tt=term, *et=term+tsz; for (const cmptype_t *fnt=fn; (tt < et) && (*tt == *fnt); tt++, fnt++); @@ -127,8 +111,7 @@ UTF8StringFieldSearcherBase::matchTermSuffix(const FieldRef & f, QueryTerm & qt) TokenizeReader reader(reinterpret_cast<const byte *> (f.data()), f.size(), dstbuf); while ( reader.hasNext() ) { - tokenize(reader); - size_t tokenlen = reader.complete(); + size_t tokenlen = reader.tokenize(normalize_mode()); if (matchTermSuffix(term, tsz, dstbuf, tokenlen)) { addHit(qt, words); } diff --git a/streamingvisitors/src/vespa/vsm/searcher/utf8stringfieldsearcherbase.h b/streamingvisitors/src/vespa/vsm/searcher/utf8stringfieldsearcherbase.h index b196f2795a4..c217a7b8866 100644 --- a/streamingvisitors/src/vespa/vsm/searcher/utf8stringfieldsearcherbase.h +++ b/streamingvisitors/src/vespa/vsm/searcher/utf8stringfieldsearcherbase.h @@ -60,9 +60,6 @@ public: protected: SharedSearcherBuf _buf; - template<typename Reader> - void tokenize(Reader & reader); - /** * Matches the given query term against the words in the given field reference * using exact or prefix match strategy. diff --git a/streamingvisitors/src/vespa/vsm/searcher/utf8suffixstringfieldsearcher.cpp b/streamingvisitors/src/vespa/vsm/searcher/utf8suffixstringfieldsearcher.cpp index 8bbacf168cf..d5bf4e4238a 100644 --- a/streamingvisitors/src/vespa/vsm/searcher/utf8suffixstringfieldsearcher.cpp +++ b/streamingvisitors/src/vespa/vsm/searcher/utf8suffixstringfieldsearcher.cpp @@ -26,8 +26,7 @@ UTF8SuffixStringFieldSearcher::matchTerms(const FieldRef & f, size_t mintsz) TokenizeReader reader(reinterpret_cast<const byte *> (f.data()), f.size(), dstbuf); while ( reader.hasNext() ) { - tokenize(reader); - size_t tokenlen = reader.complete(); + size_t tokenlen = reader.tokenize(normalize_mode()); for (auto qt : _qtl) { const cmptype_t * term; termsize_t tsz = qt->term(term); |