From 7f619ff0e43708610fd00bb953ecef723c7bb16c Mon Sep 17 00:00:00 2001 From: Tamo Date: Mon, 22 May 2023 10:33:49 +0200 Subject: [PATCH] get rids of the now unused soft_deletion_used parameter --- milli/src/update/delete_documents.rs | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/milli/src/update/delete_documents.rs b/milli/src/update/delete_documents.rs index 311f93f8f..10649b1bb 100644 --- a/milli/src/update/delete_documents.rs +++ b/milli/src/update/delete_documents.rs @@ -72,7 +72,6 @@ impl std::fmt::Display for DeletionStrategy { pub(crate) struct DetailedDocumentDeletionResult { pub deleted_documents: u64, pub remaining_documents: u64, - pub soft_deletion_used: bool, } impl<'t, 'u, 'i> DeleteDocuments<'t, 'u, 'i> { @@ -109,11 +108,8 @@ impl<'t, 'u, 'i> DeleteDocuments<'t, 'u, 'i> { Some(docid) } pub fn execute(self) -> Result { - let DetailedDocumentDeletionResult { - deleted_documents, - remaining_documents, - soft_deletion_used: _, - } = self.execute_inner()?; + let DetailedDocumentDeletionResult { deleted_documents, remaining_documents } = + self.execute_inner()?; Ok(DocumentDeletionResult { deleted_documents, remaining_documents }) } @@ -134,7 +130,6 @@ impl<'t, 'u, 'i> DeleteDocuments<'t, 'u, 'i> { return Ok(DetailedDocumentDeletionResult { deleted_documents: 0, remaining_documents: 0, - soft_deletion_used: false, }); } @@ -150,7 +145,6 @@ impl<'t, 'u, 'i> DeleteDocuments<'t, 'u, 'i> { return Ok(DetailedDocumentDeletionResult { deleted_documents: current_documents_ids_len, remaining_documents, - soft_deletion_used: false, }); } @@ -219,7 +213,6 @@ impl<'t, 'u, 'i> DeleteDocuments<'t, 'u, 'i> { return Ok(DetailedDocumentDeletionResult { deleted_documents: self.to_delete_docids.len(), remaining_documents: documents_ids.len(), - soft_deletion_used: true, }); } @@ -472,7 +465,6 @@ impl<'t, 'u, 'i> DeleteDocuments<'t, 'u, 'i> { Ok(DetailedDocumentDeletionResult { deleted_documents: self.to_delete_docids.len(), remaining_documents: documents_ids.len(), - soft_deletion_used: false, }) }