From a43720f77907d68a173db1e8a9d63594062d18d0 Mon Sep 17 00:00:00 2001 From: Buqian Zheng Date: Fri, 10 Jan 2025 16:24:58 +0800 Subject: [PATCH] fix: [2.5] fix chunk cache madvise when sparse raw data is mmaped (#39146) instead of marking as not supported, ChunkedSparseFloatColumn::DataByteSize can simply use the impl of super class. pr: https://github.com/milvus-io/milvus/pull/39145 issue: #39158 Signed-off-by: Buqian Zheng --- internal/core/src/mmap/ChunkedColumn.h | 9 --------- 1 file changed, 9 deletions(-) diff --git a/internal/core/src/mmap/ChunkedColumn.h b/internal/core/src/mmap/ChunkedColumn.h index 91b3f9530b..f05d5e7699 100644 --- a/internal/core/src/mmap/ChunkedColumn.h +++ b/internal/core/src/mmap/ChunkedColumn.h @@ -282,14 +282,6 @@ class ChunkedSparseFloatColumn : public ChunkedColumnBase { std::dynamic_pointer_cast(chunk)->Dim()); } - // This is used to advice mmap prefetch, we don't currently support mmap for - // sparse float vector thus not implemented for now. - size_t - DataByteSize() const override { - PanicInfo(ErrorCode::Unsupported, - "ByteSize not supported for sparse float column"); - } - SpanBase Span(int64_t chunk_id) const override { PanicInfo(ErrorCode::Unsupported, @@ -303,7 +295,6 @@ class ChunkedSparseFloatColumn : public ChunkedColumnBase { private: int64_t dim_ = 0; - std::vector> vec_; }; template