From c45c1fadb2c464cd6de4ddddb48f356d344be0e2 Mon Sep 17 00:00:00 2001 From: congqixia Date: Wed, 14 May 2025 17:30:21 +0800 Subject: [PATCH] enhance: [AddField] Keep all binlog when loading (#41809) Related to #41726 #41736 The load field list blocks the new field from being loaded. `load_fields` shall work as hint after tiered storage support API to specifiy this behavior. Signed-off-by: Congqi Xia --- internal/querynodev2/segments/segment_loader.go | 6 ------ 1 file changed, 6 deletions(-) diff --git a/internal/querynodev2/segments/segment_loader.go b/internal/querynodev2/segments/segment_loader.go index cc05d9cfee..eb09bddb44 100644 --- a/internal/querynodev2/segments/segment_loader.go +++ b/internal/querynodev2/segments/segment_loader.go @@ -245,12 +245,6 @@ func (loader *segmentLoader) Load(ctx context.Context, return nil, nil } coll := loader.manager.Collection.Get(collectionID) - // filter field schema which need to be loaded - for _, info := range segments { - info.BinlogPaths = lo.Filter(info.GetBinlogPaths(), func(fbl *datapb.FieldBinlog, _ int) bool { - return coll.loadFields.Contain(fbl.GetFieldID()) || common.IsSystemField(fbl.GetFieldID()) - }) - } // Filter out loaded & loading segments infos := loader.prepare(ctx, segmentType, segments...)