snowykami/meilisearch
1
0
Fork 0
mirror of https://github.com/meilisearch/meilisearch.git synced 2024-11-25 11:35:05 +08:00
Code Issues Actions 3 Packages Projects Releases Wiki Activity

Fix the merge for roaring bitmap

Browse Source
This commit is contained in:
Clément Renault 2024-09-01 23:20:19 +02:00
parent b625d31c7d
commit bb885a5810
No known key found for this signature in database
GPG Key ID: F250A4C4E3AE5F5F
1 changed files with 19 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

45
milli/src/update/new/mod.rs
View File

@ -402,37 +402,28 @@ mod indexer {
del: Option<&[u8]>,
add: Option<&[u8]>,
) -> Result<Operation> {
let bitmap = match current {
Some(current_bitmap_bytes) => {
let bitmap_without_del = match del {
Some(del_bytes) => {
let del_bitmap = CboRoaringBitmapCodec::deserialize_from(del_bytes)?;
CboRoaringBitmapCodec::intersection_with_serialized(
current_bitmap_bytes,
&del_bitmap,
)?
}
None => CboRoaringBitmapCodec::deserialize_from(current_bitmap_bytes)?,
};
let current = current.map(CboRoaringBitmapCodec::deserialize_from).transpose()?;
let del = del.map(CboRoaringBitmapCodec::deserialize_from).transpose()?;
let add = add.map(CboRoaringBitmapCodec::deserialize_from).transpose()?;
match add {
Some(add_bytes) => {
let add = CboRoaringBitmapCodec::deserialize_from(add_bytes)?;
bitmap_without_del | add
}
None => bitmap_without_del,
}
}
None => match add {
Some(add_bytes) => CboRoaringBitmapCodec::deserialize_from(add_bytes)?,
None => return Ok(Operation::Ignore),
},
match (current, del, add) {
(None, None, None) => Ok(Operation::Ignore), // but it's strange
(None, None, Some(add)) => Ok(Operation::Write(add)),
(None, Some(_del), None) => Ok(Operation::Ignore), // but it's strange
(None, Some(_del), Some(add)) => Ok(Operation::Write(add)),
(Some(_current), None, None) => Ok(Operation::Ignore), // but it's strange
(Some(current), None, Some(add)) => Ok(Operation::Write(current | add)),
(Some(current), Some(del), add) => {
let output = match add {
Some(add) => (current - del) | add,
None => current - del,
};
if bitmap.is_empty() {
if output.is_empty() {
Ok(Operation::Delete)
} else {
Ok(Operation::Write(bitmap))
Ok(Operation::Write(output))
}
}
}
}