milvus-io
/
milvus
mirror of https://github.com/milvus-io/milvus.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix: [Cherry-pick] Check parse QueryInfo error before using it (#31836) (#31864) 

Cherry-pick from master
pr: #31836
See also milvus-io/milvus-sdk-go#693 #31735

Signed-off-by: Congqi Xia <congqi.xia@zilliz.com>
pull/31861/head^2
congqixia 2024-04-03 16:07:15 +08:00 committed by GitHub
parent d4f59a6059
commit b83e48ed11
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
internal/proxy/search_util.go
View File

@ -76,6 +76,9 @@ func initSearchRequest(ctx context.Context, t *searchTask, isHybrid bool) error
annsFieldName = vecFields[0].Name
}
queryInfo, offset, err := parseSearchInfo(t.request.GetSearchParams(), t.schema.CollectionSchema)
if err != nil {
return err
}
annField := typeutil.GetFieldByName(t.schema.CollectionSchema, annsFieldName)
if queryInfo.GetGroupByFieldId() != -1 && isHybrid {
return errors.New("not support search_group_by operation in the hybrid search")
@ -84,9 +87,6 @@ func initSearchRequest(ctx context.Context, t *searchTask, isHybrid bool) error
return errors.New("not support search_group_by operation based on binary vector column")
}
if err != nil {
return err
}
t.offset = offset
plan, err := planparserv2.CreateSearchPlan(t.schema.schemaHelper, t.request.Dsl, annsFieldName, queryInfo)