From 0783582e2eba52bd8b4160118bf2e6bc51f88384 Mon Sep 17 00:00:00 2001 From: Jiquan Long Date: Fri, 10 May 2024 14:19:45 +0800 Subject: [PATCH] fix: temporarily disable two-phase retrieval when lru is enabled (#32927) issue: #31822 Signed-off-by: longjiquan --- internal/querynodev2/segments/retrieve.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/internal/querynodev2/segments/retrieve.go b/internal/querynodev2/segments/retrieve.go index 1b28ffffaa..a6ad643b23 100644 --- a/internal/querynodev2/segments/retrieve.go +++ b/internal/querynodev2/segments/retrieve.go @@ -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 {