Skip to content

Commit 5c7055d

Browse files
committed
fix
1 parent 5a46a4a commit 5c7055d

2 files changed

Lines changed: 8 additions & 5 deletions

File tree

src/paimon/global_index/lucene/jieba_analyzer.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ class JiebaTokenizer : public Lucene::Tokenizer {
7272

7373
class JiebaAnalyzer : public Lucene::Analyzer {
7474
public:
75-
JiebaAnalyzer(const JiebaTokenizerContext& context) : context_(context) {}
75+
explicit JiebaAnalyzer(const JiebaTokenizerContext& context) : context_(context) {}
7676

7777
~JiebaAnalyzer() override = default;
7878

src/paimon/global_index/lucene/lucene_api_test.cpp

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,13 @@ class LuceneInterfaceTest : public ::testing::Test {
4141
return doc_id;
4242
}
4343
int32_t docID() override {
44+
if (cursor_ >= ids_.size()) {
45+
return Lucene::DocIdSetIterator::NO_MORE_DOCS;
46+
}
4447
return ids_[cursor_];
4548
}
4649
int32_t nextDoc() override {
47-
if (cursor_ == ids_.size()) {
50+
if (cursor_ >= ids_.size()) {
4851
return Lucene::DocIdSetIterator::NO_MORE_DOCS;
4952
}
5053
return ids_[cursor_++];
@@ -163,14 +166,14 @@ class LuceneInterfaceTest : public ::testing::Test {
163166
}
164167
ASSERT_EQ(expected_doc_id_vec.size(), results->scoreDocs.size());
165168

166-
std::vector<int32_t> resule_doc_id_vec;
169+
std::vector<int32_t> result_doc_id_vec;
167170
std::vector<std::wstring> result_doc_id_content_vec;
168171
for (auto score_doc : results->scoreDocs) {
169172
Lucene::DocumentPtr result_doc = context->searcher->doc(score_doc->doc);
170-
resule_doc_id_vec.push_back(score_doc->doc);
173+
result_doc_id_vec.push_back(score_doc->doc);
171174
result_doc_id_content_vec.push_back(result_doc->get(L"id"));
172175
}
173-
ASSERT_EQ(resule_doc_id_vec, expected_doc_id_vec);
176+
ASSERT_EQ(result_doc_id_vec, expected_doc_id_vec);
174177
ASSERT_EQ(result_doc_id_content_vec, expected_doc_id_content_vec);
175178
}
176179
};

0 commit comments

Comments
 (0)