snowykami/meilisearch
1
0
Fork 0
mirror of https://github.com/meilisearch/meilisearch.git synced 2024-11-22 10:07:40 +08:00
Code Issues Actions 9 Packages Projects Releases Wiki Activity

Fix issue with Replace document method when adding and deleting a document in the same batch

Browse Source
This commit is contained in:
Louis Dureuil 2024-11-13 22:09:36 +01:00
parent 8e5b1a3ec1
commit 40dd25d6b2
No known key found for this signature in database
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
crates/milli/src/update/new/indexer/document_operation.rs
View File

@ -310,10 +310,10 @@ impl MergeChanges for MergeDocumentForReplacement {
} }
Some(InnerDocOp::Deletion) => { Some(InnerDocOp::Deletion) => {
return if is_new { return if is_new {
Ok(None)
} else {
let deletion = Deletion::create(docid, external_doc); let deletion = Deletion::create(docid, external_doc);
Ok(Some(DocumentChange::Deletion(deletion))) Ok(Some(DocumentChange::Deletion(deletion)))
} else {
Ok(None)
}; };
} }
None => unreachable!("We must not have empty set of operations on a document"), None => unreachable!("We must not have empty set of operations on a document"),