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

fix: temporarily disable two-phase retrieval when lru is enabled (#32927) 

issue: #31822

Signed-off-by: longjiquan <jiquan.long@zilliz.com>
pull/32935/head
Jiquan Long 2024-05-10 14:19:45 +08:00 committed by GitHub
parent e776c807c4
commit 0783582e2e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
internal/querynodev2/segments/retrieve.go
View File

@ -47,7 +47,9 @@ func retrieveOnSegments(ctx context.Context, mgr *Manager, segments []Segment, s
}
resultCh := make(chan segmentResult, len(segments))
plan.ignoreNonPk = len(segments) > 1 && req.GetReq().GetLimit() != typeutil.Unlimited && plan.ShouldIgnoreNonPk()
// TODO(longjiquan): remove this limit after two-phase retrieval can be applied on lru-segment.
plan.ignoreNonPk = !paramtable.Get().QueryNodeCfg.UseStreamComputing.GetAsBool() &&
len(segments) > 1 && req.GetReq().GetLimit() != typeutil.Unlimited && plan.ShouldIgnoreNonPk()
label := metrics.SealedSegmentLabel
if segType == commonpb.SegmentState_Growing {