Files
qdrant/lib/quantization/src/encoded_storage.rs
Luis Cossío fca67539f4 Mandatory batch impls (#9935)
* make `EncodedStorage::for_each_batch` mandatory

* make `DenseVectorStorageRead::for_each_in_dense_batch` mandatory

* make `DenseTQVectorStorage::for_each_in_dense_batch` mandatory

* make `DenseTQVectorStorage::read_dense_tq_bytes` mandatory

* make `QueryScorer::score_stored_batch` mandatory

...and implement for tq multivectors

* [AI] make `IdTrackerRead::internal_versions_batch` mandatory

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* [AI] make `IdTrackerRead::external_ids_batch` mandatory

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* [AI] make `DiskMappingsSource::resolve_internal_batch` mandatory

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
2026-07-22 09:21:48 -04:00

326 lines
9.7 KiB
Rust

use std::borrow::Cow;
#[cfg(feature = "testing")]
use std::io::{Read, Write};
#[cfg(feature = "testing")]
use std::num::NonZeroUsize;
#[cfg(feature = "testing")]
use std::path::Path;
use std::path::PathBuf;
use common::counter::hardware_counter::HardwareCounterCell;
#[cfg(feature = "testing")]
use common::fs::OneshotFile;
use common::mmap::MmapFlusher;
use common::types::PointOffsetType;
#[cfg(feature = "testing")]
use fs_err as fs;
#[cfg(feature = "testing")]
use fs_err::File;
pub trait EncodedStorage {
fn get_vector_data(&self, index: PointOffsetType) -> Cow<'_, [u8]>;
fn get_vector_data_opt(&self, index: PointOffsetType) -> Option<Cow<'_, [u8]>>;
fn for_each_batch(
&self,
offsets: &[PointOffsetType],
callback: impl FnMut(usize, Cow<'_, [u8]>),
);
fn is_in_ram_or_mmap() -> bool;
fn is_on_disk(&self) -> bool;
fn upsert_vector(
&mut self,
id: PointOffsetType,
vector: &[u8],
hw_counter: &HardwareCounterCell,
) -> std::io::Result<()>;
fn vectors_count(&self) -> usize;
fn flusher(&self) -> MmapFlusher;
fn files(&self) -> Vec<PathBuf>;
fn immutable_files(&self) -> Vec<PathBuf>;
/// Additional heap memory used by this storage beyond what's tracked in files.
/// RAM-based storages should report their in-memory data size here.
fn heap_size_bytes(&self) -> usize;
}
pub fn default_for_each_batch<E: EncodedStorage + ?Sized>(
this: &E,
offsets: &[u32],
mut callback: impl FnMut(usize, Cow<'_, [u8]>),
) {
for (index, &offset) in offsets.iter().enumerate() {
callback(index, this.get_vector_data(offset));
}
}
pub trait EncodedStorageBuilder {
type Storage: EncodedStorage;
type Error: std::fmt::Display;
fn build(self) -> Result<Self::Storage, Self::Error>;
fn push_vector_data(&mut self, other: &[u8]) -> Result<(), Self::Error>;
}
/// Validate that every encoded vector in `storage` has exactly `expected_size` bytes — the
/// per-vector size the quantizer derives from its metadata.
///
/// The scoring hot paths assume each stored vector has this exact size: the storage stride and
/// the quantizer metadata are both derived from the same vector parameters, so on consistent data
/// they always match. Verifying it once here at load time keeps that invariant guaranteed in
/// release builds, without paying for a bounds check on every score.
///
/// The storage uses a fixed stride for every vector, so inspecting the first encoded vector is
/// enough to validate all of them. An empty storage has no vector data to score, so there is
/// nothing to check.
pub(crate) fn validate_storage_vector_size(
storage: &impl EncodedStorage,
expected_size: usize,
) -> std::io::Result<()> {
if storage.vectors_count() == 0 {
return Ok(());
}
let actual_size = storage.get_vector_data(0).len();
if actual_size != expected_size {
return Err(std::io::Error::new(
std::io::ErrorKind::InvalidData,
format!(
"Quantized vector storage is inconsistent with its metadata: encoded vector size \
is {actual_size} bytes, but metadata expects {expected_size} bytes",
),
));
}
Ok(())
}
#[cfg(feature = "testing")]
pub struct TestEncodedStorage {
data: Vec<u8>,
quantized_vector_size: NonZeroUsize,
path: Option<PathBuf>,
}
#[cfg(feature = "testing")]
impl TestEncodedStorage {
pub fn from_file(path: &Path, quantized_vector_size: usize) -> std::io::Result<Self> {
let mut file = OneshotFile::open(path)?;
let mut buffer = Vec::new();
file.read_to_end(&mut buffer)?;
file.drop_cache()?;
if !buffer.len().is_multiple_of(quantized_vector_size) {
return Err(std::io::Error::new(
std::io::ErrorKind::InvalidData,
format!(
"TestEncodedStorage: buffer size ({}) not divisible by quantized_vector_size ({})",
buffer.len(),
quantized_vector_size,
),
));
}
let quantized_vector_size = NonZeroUsize::new(quantized_vector_size).ok_or_else(|| {
std::io::Error::new(
std::io::ErrorKind::InvalidInput,
"`quantized_vector_size` must be non-zero",
)
})?;
Ok(Self {
data: buffer,
quantized_vector_size,
path: Some(path.to_path_buf()),
})
}
}
#[cfg(feature = "testing")]
impl EncodedStorage for TestEncodedStorage {
fn get_vector_data(&self, index: PointOffsetType) -> Cow<'_, [u8]> {
self.get_vector_data_opt(index)
.unwrap_or(Cow::Borrowed(&[]))
}
fn get_vector_data_opt(&self, index: PointOffsetType) -> Option<Cow<'_, [u8]>> {
let start = self
.quantized_vector_size
.get()
.saturating_mul(index as usize);
let end = self
.quantized_vector_size
.get()
.saturating_mul(index as usize + 1);
Some(Cow::Borrowed(self.data.get(start..end)?))
}
fn for_each_batch(
&self,
offsets: &[PointOffsetType],
callback: impl FnMut(usize, Cow<'_, [u8]>),
) {
default_for_each_batch(self, offsets, callback);
}
fn upsert_vector(
&mut self,
id: PointOffsetType,
vector: &[u8],
_hw_counter: &HardwareCounterCell,
) -> std::io::Result<()> {
if vector.len() != self.quantized_vector_size.get() {
return Err(std::io::Error::new(
std::io::ErrorKind::InvalidInput,
format!(
"upsert_vector: payload length {} != quantized_vector_size {}",
vector.len(),
self.quantized_vector_size
),
));
}
// Skip hardware counter increment because it's a RAM storage.
let offset = id as usize * self.quantized_vector_size.get();
if id as usize >= self.vectors_count() {
self.data
.resize(offset + self.quantized_vector_size.get(), 0);
}
self.data[offset..offset + self.quantized_vector_size.get()].copy_from_slice(vector);
Ok(())
}
fn is_in_ram_or_mmap() -> bool {
true
}
fn is_on_disk(&self) -> bool {
false
}
fn vectors_count(&self) -> usize {
self.data.len() / self.quantized_vector_size.get()
}
fn flusher(&self) -> MmapFlusher {
Box::new(|| Ok(()))
}
fn files(&self) -> Vec<PathBuf> {
if let Some(ref path) = self.path {
vec![path.clone()]
} else {
vec![]
}
}
fn immutable_files(&self) -> Vec<PathBuf> {
self.files()
}
fn heap_size_bytes(&self) -> usize {
let Self {
data,
quantized_vector_size: _,
path: _,
} = self;
data.capacity()
}
}
#[cfg(feature = "testing")]
pub struct TestEncodedStorageBuilder {
data: Vec<u8>,
path: Option<PathBuf>,
quantized_vector_size: NonZeroUsize,
}
#[cfg(feature = "testing")]
impl TestEncodedStorageBuilder {
pub fn new(path: Option<&std::path::Path>, quantized_vector_size: usize) -> Self {
Self {
data: Vec::new(),
path: path.map(PathBuf::from),
quantized_vector_size: NonZeroUsize::new(quantized_vector_size).unwrap_or_else(|| {
panic!("quantized_vector_size must be non-zero");
}),
}
}
}
#[cfg(feature = "testing")]
impl EncodedStorageBuilder for TestEncodedStorageBuilder {
type Storage = TestEncodedStorage;
type Error = std::io::Error;
fn build(self) -> std::io::Result<Self::Storage> {
if let Some(path) = &self.path {
path.parent()
.ok_or_else(|| {
std::io::Error::new(
std::io::ErrorKind::InvalidInput,
"Path must have a parent directory",
)
})
.and_then(fs::create_dir_all)?;
let mut file = File::create(path)?;
file.write_all(&self.data)?;
file.sync_all()?;
}
Ok(TestEncodedStorage {
data: self.data,
quantized_vector_size: self.quantized_vector_size,
path: self.path,
})
}
fn push_vector_data(&mut self, other: &[u8]) -> std::io::Result<()> {
debug_assert_eq!(other.len(), self.quantized_vector_size.get());
self.data.extend_from_slice(other);
Ok(())
}
}
#[cfg(all(test, feature = "testing"))]
mod tests {
use super::*;
fn storage_with_stride(stride: usize, count: usize) -> TestEncodedStorage {
let mut builder = TestEncodedStorageBuilder::new(None, stride);
let vector = vec![0u8; stride];
for _ in 0..count {
builder.push_vector_data(&vector).unwrap();
}
builder.build().unwrap()
}
#[test]
fn accepts_matching_size() {
let storage = storage_with_stride(260, 4);
validate_storage_vector_size(&storage, 260).unwrap();
}
#[test]
fn rejects_mismatched_size() {
let storage = storage_with_stride(260, 4);
// Both a smaller and a larger expected size must be rejected (exact match required).
for expected_size in [130, 520] {
let err = validate_storage_vector_size(&storage, expected_size).unwrap_err();
assert_eq!(err.kind(), std::io::ErrorKind::InvalidData);
}
}
#[test]
fn skips_empty_storage() {
let storage = storage_with_stride(260, 0);
// With no stored vectors there is nothing to check, so any size is accepted.
validate_storage_vector_size(&storage, 999).unwrap();
}
}