未验证 提交 3b38f7f1 编写于 作者: O op-hunter 提交者: GitHub

#2649 Remove annoy search parameter validation (#2650)

* remove annoy search parameter validation
Signed-off-by: Ncmli <chengming.li@zilliz.com>

* fix test failed
Signed-off-by: Ncmli <chengming.li@zilliz.com>

* fix lint error
Signed-off-by: Ncmli <chengming.li@zilliz.com>

* fix lint error again
Signed-off-by: Ncmli <chengming.li@zilliz.com>
Co-authored-by: Ncmli <chengming.li@zilliz.com>
上级 d4fc48bf
......@@ -12,6 +12,7 @@ Please mark all change in change log and use the issue from GitHub
- \#2598 fix Milvus docker image report illegal instruction
- \#2617 Fix HNSW and RNSG index files size
- \#2637 Suit the range of HNSW parameters
- \#2649 search parameter of annoy has conflict with document
## Feature
......
......@@ -316,8 +316,8 @@ ValidationUtil::ValidateSearchParams(const milvus::json& search_params,
break;
}
case (int32_t)engine::EngineType::ANNOY: {
auto status = CheckParameterRange(search_params, knowhere::IndexParams::search_k, topk,
std::numeric_limits<int64_t>::max());
auto status = CheckParameterRange(search_params, knowhere::IndexParams::search_k,
std::numeric_limits<int64_t>::min(), std::numeric_limits<int64_t>::max());
if (!status.ok()) {
return status;
}
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册