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 <congqi.xia@zilliz.com>
This commit is contained in:
congqixia 2025-05-14 17:30:21 +08:00 committed by GitHub
parent b0f4b904cd
commit c45c1fadb2
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -245,12 +245,6 @@ func (loader *segmentLoader) Load(ctx context.Context,
return nil, nil return nil, nil
} }
coll := loader.manager.Collection.Get(collectionID) 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 // Filter out loaded & loading segments
infos := loader.prepare(ctx, segmentType, segments...) infos := loader.prepare(ctx, segmentType, segments...)