mirror of
https://github.com/meilisearch/MeiliSearch
synced 2024-12-26 22:50:07 +01:00
1170 lines
49 KiB
Rust
1170 lines
49 KiB
Rust
use std::collections::btree_map::Entry;
|
|
use std::collections::{HashMap, HashSet};
|
|
|
|
use fst::IntoStreamer;
|
|
use heed::types::{ByteSlice, DecodeIgnore, Str};
|
|
use heed::Database;
|
|
use roaring::RoaringBitmap;
|
|
use serde::{Deserialize, Serialize};
|
|
use time::OffsetDateTime;
|
|
|
|
use super::facet::delete::FacetsDelete;
|
|
use super::ClearDocuments;
|
|
use crate::error::InternalError;
|
|
use crate::facet::FacetType;
|
|
use crate::heed_codec::facet::FieldDocIdFacetCodec;
|
|
use crate::heed_codec::CboRoaringBitmapCodec;
|
|
use crate::{
|
|
ExternalDocumentsIds, FieldId, FieldIdMapMissingEntry, Index, Result, RoaringBitmapCodec,
|
|
SmallString32, BEU32,
|
|
};
|
|
|
|
pub struct DeleteDocuments<'t, 'u, 'i> {
|
|
wtxn: &'t mut heed::RwTxn<'i, 'u>,
|
|
index: &'i Index,
|
|
external_documents_ids: ExternalDocumentsIds<'static>,
|
|
to_delete_docids: RoaringBitmap,
|
|
strategy: DeletionStrategy,
|
|
}
|
|
|
|
/// Result of a [`DeleteDocuments`] operation.
|
|
#[derive(Debug, Clone, PartialEq, Eq, Serialize, Deserialize)]
|
|
pub struct DocumentDeletionResult {
|
|
pub deleted_documents: u64,
|
|
pub remaining_documents: u64,
|
|
}
|
|
|
|
/// Strategy for deleting documents.
|
|
///
|
|
/// - Soft-deleted documents are simply marked as deleted without being actually removed from DB.
|
|
/// - Hard-deleted documents are definitely suppressed from the DB.
|
|
///
|
|
/// Soft-deleted documents trade disk space for runtime performance.
|
|
///
|
|
/// Note that any of these variants can be used at any given moment for any indexation in a database.
|
|
/// For instance, you can use an [`AlwaysSoft`] followed by an [`AlwaysHard`] option without issue.
|
|
#[derive(Debug, Clone, Copy, PartialEq, Eq, Hash, Default)]
|
|
pub enum DeletionStrategy {
|
|
#[default]
|
|
/// Definitely suppress documents according to the number or size of soft-deleted documents
|
|
Dynamic,
|
|
/// Never definitely suppress documents
|
|
AlwaysSoft,
|
|
/// Always definitely suppress documents
|
|
AlwaysHard,
|
|
}
|
|
|
|
impl std::fmt::Display for DeletionStrategy {
|
|
fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
|
|
match self {
|
|
DeletionStrategy::Dynamic => write!(f, "dynamic"),
|
|
DeletionStrategy::AlwaysSoft => write!(f, "always_soft"),
|
|
DeletionStrategy::AlwaysHard => write!(f, "always_hard"),
|
|
}
|
|
}
|
|
}
|
|
|
|
/// Result of a [`DeleteDocuments`] operation, used for internal purposes.
|
|
///
|
|
/// It is a superset of the [`DocumentDeletionResult`] structure, giving
|
|
/// additional information about the algorithm used to delete the documents.
|
|
#[derive(Debug)]
|
|
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> {
|
|
pub fn new(
|
|
wtxn: &'t mut heed::RwTxn<'i, 'u>,
|
|
index: &'i Index,
|
|
) -> Result<DeleteDocuments<'t, 'u, 'i>> {
|
|
let external_documents_ids = index.external_documents_ids(wtxn)?.into_static();
|
|
|
|
Ok(DeleteDocuments {
|
|
wtxn,
|
|
index,
|
|
external_documents_ids,
|
|
to_delete_docids: RoaringBitmap::new(),
|
|
strategy: Default::default(),
|
|
})
|
|
}
|
|
|
|
pub fn strategy(&mut self, strategy: DeletionStrategy) {
|
|
self.strategy = strategy;
|
|
}
|
|
|
|
pub fn delete_document(&mut self, docid: u32) {
|
|
self.to_delete_docids.insert(docid);
|
|
}
|
|
|
|
pub fn delete_documents(&mut self, docids: &RoaringBitmap) {
|
|
self.to_delete_docids |= docids;
|
|
}
|
|
|
|
pub fn delete_external_id(&mut self, external_id: &str) -> Option<u32> {
|
|
let docid = self.external_documents_ids.get(external_id)?;
|
|
self.delete_document(docid);
|
|
Some(docid)
|
|
}
|
|
pub fn execute(self) -> Result<DocumentDeletionResult> {
|
|
let DetailedDocumentDeletionResult {
|
|
deleted_documents,
|
|
remaining_documents,
|
|
soft_deletion_used: _,
|
|
} = self.execute_inner()?;
|
|
|
|
Ok(DocumentDeletionResult { deleted_documents, remaining_documents })
|
|
}
|
|
pub(crate) fn execute_inner(mut self) -> Result<DetailedDocumentDeletionResult> {
|
|
self.index.set_updated_at(self.wtxn, &OffsetDateTime::now_utc())?;
|
|
|
|
// We retrieve the current documents ids that are in the database.
|
|
let mut documents_ids = self.index.documents_ids(self.wtxn)?;
|
|
let mut soft_deleted_docids = self.index.soft_deleted_documents_ids(self.wtxn)?;
|
|
let current_documents_ids_len = documents_ids.len();
|
|
|
|
// We can and must stop removing documents in a database that is empty.
|
|
if documents_ids.is_empty() {
|
|
// but if there was still documents to delete we clear the database entirely
|
|
if !soft_deleted_docids.is_empty() {
|
|
ClearDocuments::new(self.wtxn, self.index).execute()?;
|
|
}
|
|
return Ok(DetailedDocumentDeletionResult {
|
|
deleted_documents: 0,
|
|
remaining_documents: 0,
|
|
soft_deletion_used: false,
|
|
});
|
|
}
|
|
|
|
// We remove the documents ids that we want to delete
|
|
// from the documents in the database and write them back.
|
|
documents_ids -= &self.to_delete_docids;
|
|
self.index.put_documents_ids(self.wtxn, &documents_ids)?;
|
|
|
|
// We can execute a ClearDocuments operation when the number of documents
|
|
// to delete is exactly the number of documents in the database.
|
|
if current_documents_ids_len == self.to_delete_docids.len() {
|
|
let remaining_documents = ClearDocuments::new(self.wtxn, self.index).execute()?;
|
|
return Ok(DetailedDocumentDeletionResult {
|
|
deleted_documents: current_documents_ids_len,
|
|
remaining_documents,
|
|
soft_deletion_used: false,
|
|
});
|
|
}
|
|
|
|
let fields_ids_map = self.index.fields_ids_map(self.wtxn)?;
|
|
let mut field_distribution = self.index.field_distribution(self.wtxn)?;
|
|
|
|
// we update the field distribution
|
|
for docid in self.to_delete_docids.iter() {
|
|
let key = BEU32::new(docid);
|
|
let document =
|
|
self.index.documents.get(self.wtxn, &key)?.ok_or(
|
|
InternalError::DatabaseMissingEntry { db_name: "documents", key: None },
|
|
)?;
|
|
for (fid, _value) in document.iter() {
|
|
let field_name =
|
|
fields_ids_map.name(fid).ok_or(FieldIdMapMissingEntry::FieldId {
|
|
field_id: fid,
|
|
process: "delete documents",
|
|
})?;
|
|
if let Entry::Occupied(mut entry) = field_distribution.entry(field_name.to_string())
|
|
{
|
|
match entry.get().checked_sub(1) {
|
|
Some(0) | None => entry.remove(),
|
|
Some(count) => entry.insert(count),
|
|
};
|
|
}
|
|
}
|
|
}
|
|
|
|
self.index.put_field_distribution(self.wtxn, &field_distribution)?;
|
|
|
|
soft_deleted_docids |= &self.to_delete_docids;
|
|
|
|
// We always soft-delete the documents, even if they will be permanently
|
|
// deleted immediately after.
|
|
self.index.put_soft_deleted_documents_ids(self.wtxn, &soft_deleted_docids)?;
|
|
|
|
// decide for a hard or soft deletion depending on the strategy
|
|
let soft_deletion = match self.strategy {
|
|
DeletionStrategy::Dynamic => {
|
|
// decide to keep the soft deleted in the DB for now if they meet 2 criteria:
|
|
// 1. There is less than a fixed rate of 50% of soft-deleted to actual documents, *and*
|
|
// 2. Soft-deleted occupy an average of less than a fixed size on disk
|
|
|
|
let size_used = self.index.used_size()?;
|
|
let nb_documents = self.index.number_of_documents(self.wtxn)?;
|
|
let nb_soft_deleted = soft_deleted_docids.len();
|
|
|
|
(nb_soft_deleted < nb_documents) && {
|
|
const SOFT_DELETED_SIZE_BYTE_THRESHOLD: u64 = 1_073_741_824; // 1GiB
|
|
|
|
// nb_documents + nb_soft_deleted !=0 because if nb_documents is 0 we short-circuit earlier, and then we moved the documents to delete
|
|
// from the documents_docids to the soft_deleted_docids.
|
|
let estimated_document_size = size_used / (nb_documents + nb_soft_deleted);
|
|
let estimated_size_used_by_soft_deleted =
|
|
estimated_document_size * nb_soft_deleted;
|
|
estimated_size_used_by_soft_deleted < SOFT_DELETED_SIZE_BYTE_THRESHOLD
|
|
}
|
|
}
|
|
DeletionStrategy::AlwaysSoft => true,
|
|
DeletionStrategy::AlwaysHard => false,
|
|
};
|
|
|
|
if soft_deletion {
|
|
// Keep the soft-deleted in the DB
|
|
return Ok(DetailedDocumentDeletionResult {
|
|
deleted_documents: self.to_delete_docids.len(),
|
|
remaining_documents: documents_ids.len(),
|
|
soft_deletion_used: true,
|
|
});
|
|
}
|
|
|
|
self.to_delete_docids = soft_deleted_docids;
|
|
|
|
let Index {
|
|
env: _env,
|
|
main: _main,
|
|
word_docids,
|
|
exact_word_docids,
|
|
word_prefix_docids,
|
|
exact_word_prefix_docids,
|
|
docid_word_positions,
|
|
word_pair_proximity_docids,
|
|
field_id_word_count_docids,
|
|
word_prefix_pair_proximity_docids,
|
|
prefix_word_pair_proximity_docids,
|
|
word_position_docids,
|
|
word_prefix_position_docids,
|
|
facet_id_f64_docids: _,
|
|
facet_id_string_docids: _,
|
|
field_id_docid_facet_f64s: _,
|
|
field_id_docid_facet_strings: _,
|
|
facet_id_exists_docids,
|
|
documents,
|
|
} = self.index;
|
|
|
|
// Retrieve the words contained in the documents.
|
|
let mut words = Vec::new();
|
|
for docid in &self.to_delete_docids {
|
|
documents.delete(self.wtxn, &BEU32::new(docid))?;
|
|
|
|
// We iterate through the words positions of the document id, retrieve the word and delete the positions.
|
|
// We create an iterator to be able to get the content and delete the key-value itself.
|
|
// It's faster to acquire a cursor to get and delete, as we avoid traversing the LMDB B-Tree two times but only once.
|
|
let mut iter = docid_word_positions.prefix_iter_mut(self.wtxn, &(docid, ""))?;
|
|
while let Some(result) = iter.next() {
|
|
let ((_docid, word), _positions) = result?;
|
|
// This boolean will indicate if we must remove this word from the words FST.
|
|
words.push((SmallString32::from(word), false));
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.del_current()? };
|
|
}
|
|
}
|
|
// We acquire the current external documents ids map...
|
|
// Note that its soft-deleted document ids field will be equal to the `to_delete_docids`
|
|
let mut new_external_documents_ids = self.index.external_documents_ids(self.wtxn)?;
|
|
// We then remove the soft-deleted docids from it
|
|
new_external_documents_ids.delete_soft_deleted_documents_ids_from_fsts()?;
|
|
// and write it back to the main database.
|
|
let new_external_documents_ids = new_external_documents_ids.into_static();
|
|
self.index.put_external_documents_ids(self.wtxn, &new_external_documents_ids)?;
|
|
|
|
// Maybe we can improve the get performance of the words
|
|
// if we sort the words first, keeping the LMDB pages in cache.
|
|
words.sort_unstable();
|
|
|
|
// We iterate over the words and delete the documents ids
|
|
// from the word docids database.
|
|
for (word, must_remove) in &mut words {
|
|
remove_from_word_docids(
|
|
self.wtxn,
|
|
word_docids,
|
|
word.as_str(),
|
|
must_remove,
|
|
&self.to_delete_docids,
|
|
)?;
|
|
|
|
remove_from_word_docids(
|
|
self.wtxn,
|
|
exact_word_docids,
|
|
word.as_str(),
|
|
must_remove,
|
|
&self.to_delete_docids,
|
|
)?;
|
|
}
|
|
|
|
// We construct an FST set that contains the words to delete from the words FST.
|
|
let words_to_delete =
|
|
words.iter().filter_map(
|
|
|(word, must_remove)| {
|
|
if *must_remove {
|
|
Some(word.as_str())
|
|
} else {
|
|
None
|
|
}
|
|
},
|
|
);
|
|
let words_to_delete = fst::Set::from_iter(words_to_delete)?;
|
|
|
|
let new_words_fst = {
|
|
// We retrieve the current words FST from the database.
|
|
let words_fst = self.index.words_fst(self.wtxn)?;
|
|
let difference = words_fst.op().add(&words_to_delete).difference();
|
|
|
|
// We stream the new external ids that does no more contains the to-delete external ids.
|
|
let mut new_words_fst_builder = fst::SetBuilder::memory();
|
|
new_words_fst_builder.extend_stream(difference.into_stream())?;
|
|
|
|
// We create an words FST set from the above builder.
|
|
new_words_fst_builder.into_set()
|
|
};
|
|
|
|
// We write the new words FST into the main database.
|
|
self.index.put_words_fst(self.wtxn, &new_words_fst)?;
|
|
|
|
let prefixes_to_delete =
|
|
remove_from_word_prefix_docids(self.wtxn, word_prefix_docids, &self.to_delete_docids)?;
|
|
|
|
let exact_prefix_to_delete = remove_from_word_prefix_docids(
|
|
self.wtxn,
|
|
exact_word_prefix_docids,
|
|
&self.to_delete_docids,
|
|
)?;
|
|
|
|
let all_prefixes_to_delete = prefixes_to_delete.op().add(&exact_prefix_to_delete).union();
|
|
|
|
// We compute the new prefix FST and write it only if there is a change.
|
|
if !prefixes_to_delete.is_empty() || !exact_prefix_to_delete.is_empty() {
|
|
let new_words_prefixes_fst = {
|
|
// We retrieve the current words prefixes FST from the database.
|
|
let words_prefixes_fst = self.index.words_prefixes_fst(self.wtxn)?;
|
|
let difference =
|
|
words_prefixes_fst.op().add(all_prefixes_to_delete.into_stream()).difference();
|
|
|
|
// We stream the new external ids that does no more contains the to-delete external ids.
|
|
let mut new_words_prefixes_fst_builder = fst::SetBuilder::memory();
|
|
new_words_prefixes_fst_builder.extend_stream(difference.into_stream())?;
|
|
|
|
// We create an words FST set from the above builder.
|
|
new_words_prefixes_fst_builder.into_set()
|
|
};
|
|
|
|
// We write the new words prefixes FST into the main database.
|
|
self.index.put_words_prefixes_fst(self.wtxn, &new_words_prefixes_fst)?;
|
|
}
|
|
|
|
for db in [word_prefix_pair_proximity_docids, prefix_word_pair_proximity_docids] {
|
|
// We delete the documents ids from the word prefix pair proximity database docids
|
|
// and remove the empty pairs too.
|
|
let db = db.remap_key_type::<ByteSlice>();
|
|
let mut iter = db.iter_mut(self.wtxn)?;
|
|
while let Some(result) = iter.next() {
|
|
let (key, mut docids) = result?;
|
|
let previous_len = docids.len();
|
|
docids -= &self.to_delete_docids;
|
|
if docids.is_empty() {
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.del_current()? };
|
|
} else if docids.len() != previous_len {
|
|
let key = key.to_owned();
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.put_current(&key, &docids)? };
|
|
}
|
|
}
|
|
}
|
|
|
|
// We delete the documents ids that are under the pairs of words,
|
|
// it is faster and use no memory to iterate over all the words pairs than
|
|
// to compute the cartesian product of every words of the deleted documents.
|
|
let mut iter =
|
|
word_pair_proximity_docids.remap_key_type::<ByteSlice>().iter_mut(self.wtxn)?;
|
|
while let Some(result) = iter.next() {
|
|
let (bytes, mut docids) = result?;
|
|
let previous_len = docids.len();
|
|
docids -= &self.to_delete_docids;
|
|
if docids.is_empty() {
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.del_current()? };
|
|
} else if docids.len() != previous_len {
|
|
let bytes = bytes.to_owned();
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.put_current(&bytes, &docids)? };
|
|
}
|
|
}
|
|
|
|
drop(iter);
|
|
|
|
// We delete the documents ids that are under the word level position docids.
|
|
let mut iter = word_position_docids.iter_mut(self.wtxn)?.remap_key_type::<ByteSlice>();
|
|
while let Some(result) = iter.next() {
|
|
let (bytes, mut docids) = result?;
|
|
let previous_len = docids.len();
|
|
docids -= &self.to_delete_docids;
|
|
if docids.is_empty() {
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.del_current()? };
|
|
} else if docids.len() != previous_len {
|
|
let bytes = bytes.to_owned();
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.put_current(&bytes, &docids)? };
|
|
}
|
|
}
|
|
|
|
drop(iter);
|
|
|
|
// We delete the documents ids that are under the word prefix level position docids.
|
|
let mut iter =
|
|
word_prefix_position_docids.iter_mut(self.wtxn)?.remap_key_type::<ByteSlice>();
|
|
while let Some(result) = iter.next() {
|
|
let (bytes, mut docids) = result?;
|
|
let previous_len = docids.len();
|
|
docids -= &self.to_delete_docids;
|
|
if docids.is_empty() {
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.del_current()? };
|
|
} else if docids.len() != previous_len {
|
|
let bytes = bytes.to_owned();
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.put_current(&bytes, &docids)? };
|
|
}
|
|
}
|
|
|
|
drop(iter);
|
|
|
|
// Remove the documents ids from the field id word count database.
|
|
let mut iter = field_id_word_count_docids.iter_mut(self.wtxn)?;
|
|
while let Some((key, mut docids)) = iter.next().transpose()? {
|
|
let previous_len = docids.len();
|
|
docids -= &self.to_delete_docids;
|
|
if docids.is_empty() {
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.del_current()? };
|
|
} else if docids.len() != previous_len {
|
|
let key = key.to_owned();
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.put_current(&key, &docids)? };
|
|
}
|
|
}
|
|
|
|
drop(iter);
|
|
|
|
if let Some(mut rtree) = self.index.geo_rtree(self.wtxn)? {
|
|
let mut geo_faceted_doc_ids = self.index.geo_faceted_documents_ids(self.wtxn)?;
|
|
|
|
let (points_to_remove, docids_to_remove): (Vec<_>, RoaringBitmap) = rtree
|
|
.iter()
|
|
.filter(|&point| self.to_delete_docids.contains(point.data.0))
|
|
.cloned()
|
|
.map(|point| (point, point.data.0))
|
|
.unzip();
|
|
points_to_remove.iter().for_each(|point| {
|
|
rtree.remove(point);
|
|
});
|
|
geo_faceted_doc_ids -= docids_to_remove;
|
|
|
|
self.index.put_geo_rtree(self.wtxn, &rtree)?;
|
|
self.index.put_geo_faceted_documents_ids(self.wtxn, &geo_faceted_doc_ids)?;
|
|
}
|
|
|
|
for facet_type in [FacetType::Number, FacetType::String] {
|
|
let mut affected_facet_values = HashMap::new();
|
|
for field_id in self.index.faceted_fields_ids(self.wtxn)? {
|
|
// Remove docids from the number faceted documents ids
|
|
let mut docids =
|
|
self.index.faceted_documents_ids(self.wtxn, field_id, facet_type)?;
|
|
docids -= &self.to_delete_docids;
|
|
self.index.put_faceted_documents_ids(self.wtxn, field_id, facet_type, &docids)?;
|
|
|
|
let facet_values = remove_docids_from_field_id_docid_facet_value(
|
|
self.index,
|
|
self.wtxn,
|
|
facet_type,
|
|
field_id,
|
|
&self.to_delete_docids,
|
|
)?;
|
|
if !facet_values.is_empty() {
|
|
affected_facet_values.insert(field_id, facet_values);
|
|
}
|
|
}
|
|
FacetsDelete::new(
|
|
self.index,
|
|
facet_type,
|
|
affected_facet_values,
|
|
&self.to_delete_docids,
|
|
)
|
|
.execute(self.wtxn)?;
|
|
}
|
|
|
|
// We delete the documents ids that are under the facet field id values.
|
|
remove_docids_from_facet_id_exists_docids(
|
|
self.wtxn,
|
|
facet_id_exists_docids,
|
|
&self.to_delete_docids,
|
|
)?;
|
|
|
|
self.index.put_soft_deleted_documents_ids(self.wtxn, &RoaringBitmap::new())?;
|
|
|
|
Ok(DetailedDocumentDeletionResult {
|
|
deleted_documents: self.to_delete_docids.len(),
|
|
remaining_documents: documents_ids.len(),
|
|
soft_deletion_used: false,
|
|
})
|
|
}
|
|
}
|
|
|
|
fn remove_from_word_prefix_docids(
|
|
txn: &mut heed::RwTxn,
|
|
db: &Database<Str, RoaringBitmapCodec>,
|
|
to_remove: &RoaringBitmap,
|
|
) -> Result<fst::Set<Vec<u8>>> {
|
|
let mut prefixes_to_delete = fst::SetBuilder::memory();
|
|
|
|
// We iterate over the word prefix docids database and remove the deleted documents ids
|
|
// from every docids lists. We register the empty prefixes in an fst Set for futur deletion.
|
|
let mut iter = db.iter_mut(txn)?;
|
|
while let Some(result) = iter.next() {
|
|
let (prefix, mut docids) = result?;
|
|
let prefix = prefix.to_owned();
|
|
let previous_len = docids.len();
|
|
docids -= to_remove;
|
|
if docids.is_empty() {
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.del_current()? };
|
|
prefixes_to_delete.insert(prefix)?;
|
|
} else if docids.len() != previous_len {
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.put_current(&prefix, &docids)? };
|
|
}
|
|
}
|
|
|
|
Ok(prefixes_to_delete.into_set())
|
|
}
|
|
|
|
fn remove_from_word_docids(
|
|
txn: &mut heed::RwTxn,
|
|
db: &heed::Database<Str, RoaringBitmapCodec>,
|
|
word: &str,
|
|
must_remove: &mut bool,
|
|
to_remove: &RoaringBitmap,
|
|
) -> Result<()> {
|
|
// We create an iterator to be able to get the content and delete the word docids.
|
|
// It's faster to acquire a cursor to get and delete or put, as we avoid traversing
|
|
// the LMDB B-Tree two times but only once.
|
|
let mut iter = db.prefix_iter_mut(txn, word)?;
|
|
if let Some((key, mut docids)) = iter.next().transpose()? {
|
|
if key == word {
|
|
let previous_len = docids.len();
|
|
docids -= to_remove;
|
|
if docids.is_empty() {
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.del_current()? };
|
|
*must_remove = true;
|
|
} else if docids.len() != previous_len {
|
|
let key = key.to_owned();
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.put_current(&key, &docids)? };
|
|
}
|
|
}
|
|
}
|
|
|
|
Ok(())
|
|
}
|
|
|
|
fn remove_docids_from_field_id_docid_facet_value(
|
|
index: &'_ Index,
|
|
wtxn: &'_ mut heed::RwTxn,
|
|
facet_type: FacetType,
|
|
field_id: FieldId,
|
|
to_remove: &RoaringBitmap,
|
|
) -> heed::Result<HashSet<Vec<u8>>> {
|
|
let db = match facet_type {
|
|
FacetType::String => {
|
|
index.field_id_docid_facet_strings.remap_types::<ByteSlice, DecodeIgnore>()
|
|
}
|
|
FacetType::Number => {
|
|
index.field_id_docid_facet_f64s.remap_types::<ByteSlice, DecodeIgnore>()
|
|
}
|
|
};
|
|
let mut all_affected_facet_values = HashSet::default();
|
|
let mut iter = db
|
|
.prefix_iter_mut(wtxn, &field_id.to_be_bytes())?
|
|
.remap_key_type::<FieldDocIdFacetCodec<ByteSlice>>();
|
|
|
|
while let Some(result) = iter.next() {
|
|
let ((_, docid, facet_value), _) = result?;
|
|
if to_remove.contains(docid) {
|
|
if !all_affected_facet_values.contains(facet_value) {
|
|
all_affected_facet_values.insert(facet_value.to_owned());
|
|
}
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.del_current()? };
|
|
}
|
|
}
|
|
|
|
Ok(all_affected_facet_values)
|
|
}
|
|
|
|
fn remove_docids_from_facet_id_exists_docids<'a, C>(
|
|
wtxn: &'a mut heed::RwTxn,
|
|
db: &heed::Database<C, CboRoaringBitmapCodec>,
|
|
to_remove: &RoaringBitmap,
|
|
) -> heed::Result<()>
|
|
where
|
|
C: heed::BytesDecode<'a> + heed::BytesEncode<'a>,
|
|
{
|
|
let mut iter = db.remap_key_type::<ByteSlice>().iter_mut(wtxn)?;
|
|
while let Some(result) = iter.next() {
|
|
let (bytes, mut docids) = result?;
|
|
let previous_len = docids.len();
|
|
docids -= to_remove;
|
|
if docids.is_empty() {
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.del_current()? };
|
|
} else if docids.len() != previous_len {
|
|
let bytes = bytes.to_owned();
|
|
// safety: we don't keep references from inside the LMDB database.
|
|
unsafe { iter.put_current(&bytes, &docids)? };
|
|
}
|
|
}
|
|
|
|
Ok(())
|
|
}
|
|
|
|
#[cfg(test)]
|
|
mod tests {
|
|
use big_s::S;
|
|
use heed::RwTxn;
|
|
use maplit::hashset;
|
|
|
|
use super::*;
|
|
use crate::index::tests::TempIndex;
|
|
use crate::{db_snap, Filter};
|
|
|
|
fn delete_documents<'t>(
|
|
wtxn: &mut RwTxn<'t, '_>,
|
|
index: &'t Index,
|
|
external_ids: &[&str],
|
|
strategy: DeletionStrategy,
|
|
) -> Vec<u32> {
|
|
let external_document_ids = index.external_documents_ids(wtxn).unwrap();
|
|
let ids_to_delete: Vec<u32> = external_ids
|
|
.iter()
|
|
.map(|id| external_document_ids.get(id.as_bytes()).unwrap())
|
|
.collect();
|
|
|
|
// Delete some documents.
|
|
let mut builder = DeleteDocuments::new(wtxn, index).unwrap();
|
|
builder.strategy(strategy);
|
|
external_ids.iter().for_each(|id| {
|
|
builder.delete_external_id(id);
|
|
});
|
|
builder.execute().unwrap();
|
|
|
|
ids_to_delete
|
|
}
|
|
|
|
fn delete_documents_with_numbers_as_primary_key_(deletion_strategy: DeletionStrategy) {
|
|
let index = TempIndex::new();
|
|
|
|
let mut wtxn = index.write_txn().unwrap();
|
|
index
|
|
.add_documents_using_wtxn(
|
|
&mut wtxn,
|
|
documents!([
|
|
{ "id": 0, "name": "kevin", "object": { "key1": "value1", "key2": "value2" } },
|
|
{ "id": 1, "name": "kevina", "array": ["I", "am", "fine"] },
|
|
{ "id": 2, "name": "benoit", "array_of_object": [{ "wow": "amazing" }] }
|
|
]),
|
|
)
|
|
.unwrap();
|
|
|
|
// delete those documents, ids are synchronous therefore 0, 1, and 2.
|
|
let mut builder = DeleteDocuments::new(&mut wtxn, &index).unwrap();
|
|
builder.delete_document(0);
|
|
builder.delete_document(1);
|
|
builder.delete_document(2);
|
|
builder.strategy(deletion_strategy);
|
|
builder.execute().unwrap();
|
|
|
|
wtxn.commit().unwrap();
|
|
|
|
// All these snapshots should be empty since the database was cleared
|
|
db_snap!(index, documents_ids, deletion_strategy);
|
|
db_snap!(index, word_docids, deletion_strategy);
|
|
db_snap!(index, word_pair_proximity_docids, deletion_strategy);
|
|
db_snap!(index, facet_id_exists_docids, deletion_strategy);
|
|
db_snap!(index, soft_deleted_documents_ids, deletion_strategy);
|
|
|
|
let rtxn = index.read_txn().unwrap();
|
|
|
|
assert!(index.field_distribution(&rtxn).unwrap().is_empty());
|
|
}
|
|
|
|
#[test]
|
|
fn delete_documents_with_numbers_as_primary_key() {
|
|
delete_documents_with_numbers_as_primary_key_(DeletionStrategy::AlwaysHard);
|
|
delete_documents_with_numbers_as_primary_key_(DeletionStrategy::AlwaysSoft);
|
|
}
|
|
|
|
fn delete_documents_with_strange_primary_key_(strategy: DeletionStrategy) {
|
|
let index = TempIndex::new();
|
|
|
|
index
|
|
.update_settings(|settings| settings.set_searchable_fields(vec!["name".to_string()]))
|
|
.unwrap();
|
|
|
|
let mut wtxn = index.write_txn().unwrap();
|
|
index
|
|
.add_documents_using_wtxn(
|
|
&mut wtxn,
|
|
documents!([
|
|
{ "mysuperid": 0, "name": "kevin" },
|
|
{ "mysuperid": 1, "name": "kevina" },
|
|
{ "mysuperid": 2, "name": "benoit" }
|
|
]),
|
|
)
|
|
.unwrap();
|
|
wtxn.commit().unwrap();
|
|
|
|
let mut wtxn = index.write_txn().unwrap();
|
|
|
|
// Delete not all of the documents but some of them.
|
|
let mut builder = DeleteDocuments::new(&mut wtxn, &index).unwrap();
|
|
builder.delete_external_id("0");
|
|
builder.delete_external_id("1");
|
|
builder.strategy(strategy);
|
|
builder.execute().unwrap();
|
|
wtxn.commit().unwrap();
|
|
|
|
db_snap!(index, documents_ids, strategy);
|
|
db_snap!(index, word_docids, strategy);
|
|
db_snap!(index, word_pair_proximity_docids, strategy);
|
|
db_snap!(index, soft_deleted_documents_ids, strategy);
|
|
}
|
|
|
|
#[test]
|
|
fn delete_documents_with_strange_primary_key() {
|
|
delete_documents_with_strange_primary_key_(DeletionStrategy::AlwaysHard);
|
|
delete_documents_with_strange_primary_key_(DeletionStrategy::AlwaysSoft);
|
|
}
|
|
|
|
fn filtered_placeholder_search_should_not_return_deleted_documents_(
|
|
deletion_strategy: DeletionStrategy,
|
|
) {
|
|
let index = TempIndex::new();
|
|
|
|
let mut wtxn = index.write_txn().unwrap();
|
|
|
|
index
|
|
.update_settings_using_wtxn(&mut wtxn, |settings| {
|
|
settings.set_primary_key(S("docid"));
|
|
settings.set_filterable_fields(hashset! { S("label"), S("label2") });
|
|
})
|
|
.unwrap();
|
|
|
|
index
|
|
.add_documents_using_wtxn(
|
|
&mut wtxn,
|
|
documents!([
|
|
{ "docid": "1_4", "label": ["sign"] },
|
|
{ "docid": "1_5", "label": ["letter"] },
|
|
{ "docid": "1_7", "label": ["abstract","cartoon","design","pattern"] },
|
|
{ "docid": "1_36", "label": ["drawing","painting","pattern"] },
|
|
{ "docid": "1_37", "label": ["art","drawing","outdoor"] },
|
|
{ "docid": "1_38", "label": ["aquarium","art","drawing"] },
|
|
{ "docid": "1_39", "label": ["abstract"] },
|
|
{ "docid": "1_40", "label": ["cartoon"] },
|
|
{ "docid": "1_41", "label": ["art","drawing"] },
|
|
{ "docid": "1_42", "label": ["art","pattern"] },
|
|
{ "docid": "1_43", "label": ["abstract","art","drawing","pattern"] },
|
|
{ "docid": "1_44", "label": ["drawing"] },
|
|
{ "docid": "1_45", "label": ["art"] },
|
|
{ "docid": "1_46", "label": ["abstract","colorfulness","pattern"] },
|
|
{ "docid": "1_47", "label": ["abstract","pattern"] },
|
|
{ "docid": "1_52", "label": ["abstract","cartoon"] },
|
|
{ "docid": "1_57", "label": ["abstract","drawing","pattern"] },
|
|
{ "docid": "1_58", "label": ["abstract","art","cartoon"] },
|
|
{ "docid": "1_68", "label": ["design"] },
|
|
{ "docid": "1_69", "label": ["geometry"] },
|
|
{ "docid": "1_70", "label2": ["geometry", 1.2] },
|
|
{ "docid": "1_71", "label2": ["design", 2.2] },
|
|
{ "docid": "1_72", "label2": ["geometry", 1.2] }
|
|
]),
|
|
)
|
|
.unwrap();
|
|
|
|
delete_documents(&mut wtxn, &index, &["1_4", "1_70", "1_72"], deletion_strategy);
|
|
|
|
// Placeholder search with filter
|
|
let filter = Filter::from_str("label = sign").unwrap().unwrap();
|
|
let results = index.search(&wtxn).filter(filter).execute().unwrap();
|
|
assert!(results.documents_ids.is_empty());
|
|
|
|
wtxn.commit().unwrap();
|
|
|
|
db_snap!(index, soft_deleted_documents_ids, deletion_strategy);
|
|
db_snap!(index, word_docids, deletion_strategy);
|
|
db_snap!(index, facet_id_f64_docids, deletion_strategy);
|
|
db_snap!(index, word_pair_proximity_docids, deletion_strategy);
|
|
db_snap!(index, facet_id_exists_docids, deletion_strategy);
|
|
db_snap!(index, facet_id_string_docids, deletion_strategy);
|
|
}
|
|
|
|
#[test]
|
|
fn filtered_placeholder_search_should_not_return_deleted_documents() {
|
|
filtered_placeholder_search_should_not_return_deleted_documents_(
|
|
DeletionStrategy::AlwaysHard,
|
|
);
|
|
filtered_placeholder_search_should_not_return_deleted_documents_(
|
|
DeletionStrategy::AlwaysSoft,
|
|
);
|
|
}
|
|
|
|
fn placeholder_search_should_not_return_deleted_documents_(
|
|
deletion_strategy: DeletionStrategy,
|
|
) {
|
|
let index = TempIndex::new();
|
|
|
|
let mut wtxn = index.write_txn().unwrap();
|
|
index
|
|
.update_settings_using_wtxn(&mut wtxn, |settings| {
|
|
settings.set_primary_key(S("docid"));
|
|
})
|
|
.unwrap();
|
|
|
|
index
|
|
.add_documents_using_wtxn(
|
|
&mut wtxn,
|
|
documents!([
|
|
{ "docid": "1_4", "label": ["sign"] },
|
|
{ "docid": "1_5", "label": ["letter"] },
|
|
{ "docid": "1_7", "label": ["abstract","cartoon","design","pattern"] },
|
|
{ "docid": "1_36", "label": ["drawing","painting","pattern"] },
|
|
{ "docid": "1_37", "label": ["art","drawing","outdoor"] },
|
|
{ "docid": "1_38", "label": ["aquarium","art","drawing"] },
|
|
{ "docid": "1_39", "label": ["abstract"] },
|
|
{ "docid": "1_40", "label": ["cartoon"] },
|
|
{ "docid": "1_41", "label": ["art","drawing"] },
|
|
{ "docid": "1_42", "label": ["art","pattern"] },
|
|
{ "docid": "1_43", "label": ["abstract","art","drawing","pattern"] },
|
|
{ "docid": "1_44", "label": ["drawing"] },
|
|
{ "docid": "1_45", "label": ["art"] },
|
|
{ "docid": "1_46", "label": ["abstract","colorfulness","pattern"] },
|
|
{ "docid": "1_47", "label": ["abstract","pattern"] },
|
|
{ "docid": "1_52", "label": ["abstract","cartoon"] },
|
|
{ "docid": "1_57", "label": ["abstract","drawing","pattern"] },
|
|
{ "docid": "1_58", "label": ["abstract","art","cartoon"] },
|
|
{ "docid": "1_68", "label": ["design"] },
|
|
{ "docid": "1_69", "label": ["geometry"] },
|
|
{ "docid": "1_70", "label2": ["geometry", 1.2] },
|
|
{ "docid": "1_71", "label2": ["design", 2.2] },
|
|
{ "docid": "1_72", "label2": ["geometry", 1.2] }
|
|
]),
|
|
)
|
|
.unwrap();
|
|
|
|
let deleted_internal_ids = delete_documents(&mut wtxn, &index, &["1_4"], deletion_strategy);
|
|
|
|
// Placeholder search
|
|
let results = index.search(&wtxn).execute().unwrap();
|
|
assert!(!results.documents_ids.is_empty());
|
|
for id in results.documents_ids.iter() {
|
|
assert!(
|
|
!deleted_internal_ids.contains(id),
|
|
"The document {} was supposed to be deleted",
|
|
id
|
|
);
|
|
}
|
|
|
|
wtxn.commit().unwrap();
|
|
}
|
|
|
|
#[test]
|
|
fn placeholder_search_should_not_return_deleted_documents() {
|
|
placeholder_search_should_not_return_deleted_documents_(DeletionStrategy::AlwaysHard);
|
|
placeholder_search_should_not_return_deleted_documents_(DeletionStrategy::AlwaysSoft);
|
|
}
|
|
|
|
fn search_should_not_return_deleted_documents_(deletion_strategy: DeletionStrategy) {
|
|
let index = TempIndex::new();
|
|
|
|
let mut wtxn = index.write_txn().unwrap();
|
|
index
|
|
.update_settings_using_wtxn(&mut wtxn, |settings| {
|
|
settings.set_primary_key(S("docid"));
|
|
})
|
|
.unwrap();
|
|
|
|
index
|
|
.add_documents_using_wtxn(
|
|
&mut wtxn,
|
|
documents!([
|
|
{ "docid": "1_4", "label": ["sign"] },
|
|
{ "docid": "1_5", "label": ["letter"] },
|
|
{ "docid": "1_7", "label": ["abstract","cartoon","design","pattern"] },
|
|
{ "docid": "1_36", "label": ["drawing","painting","pattern"] },
|
|
{ "docid": "1_37", "label": ["art","drawing","outdoor"] },
|
|
{ "docid": "1_38", "label": ["aquarium","art","drawing"] },
|
|
{ "docid": "1_39", "label": ["abstract"] },
|
|
{ "docid": "1_40", "label": ["cartoon"] },
|
|
{ "docid": "1_41", "label": ["art","drawing"] },
|
|
{ "docid": "1_42", "label": ["art","pattern"] },
|
|
{ "docid": "1_43", "label": ["abstract","art","drawing","pattern"] },
|
|
{ "docid": "1_44", "label": ["drawing"] },
|
|
{ "docid": "1_45", "label": ["art"] },
|
|
{ "docid": "1_46", "label": ["abstract","colorfulness","pattern"] },
|
|
{ "docid": "1_47", "label": ["abstract","pattern"] },
|
|
{ "docid": "1_52", "label": ["abstract","cartoon"] },
|
|
{ "docid": "1_57", "label": ["abstract","drawing","pattern"] },
|
|
{ "docid": "1_58", "label": ["abstract","art","cartoon"] },
|
|
{ "docid": "1_68", "label": ["design"] },
|
|
{ "docid": "1_69", "label": ["geometry"] },
|
|
{ "docid": "1_70", "label2": ["geometry", 1.2] },
|
|
{ "docid": "1_71", "label2": ["design", 2.2] },
|
|
{ "docid": "1_72", "label2": ["geometry", 1.2] }
|
|
]),
|
|
)
|
|
.unwrap();
|
|
|
|
let deleted_internal_ids =
|
|
delete_documents(&mut wtxn, &index, &["1_7", "1_52"], deletion_strategy);
|
|
|
|
// search for abstract
|
|
let results = index.search(&wtxn).query("abstract").execute().unwrap();
|
|
assert!(!results.documents_ids.is_empty());
|
|
for id in results.documents_ids.iter() {
|
|
assert!(
|
|
!deleted_internal_ids.contains(id),
|
|
"The document {} was supposed to be deleted",
|
|
id
|
|
);
|
|
}
|
|
|
|
wtxn.commit().unwrap();
|
|
|
|
db_snap!(index, soft_deleted_documents_ids, deletion_strategy);
|
|
}
|
|
|
|
#[test]
|
|
fn search_should_not_return_deleted_documents() {
|
|
search_should_not_return_deleted_documents_(DeletionStrategy::AlwaysHard);
|
|
search_should_not_return_deleted_documents_(DeletionStrategy::AlwaysSoft);
|
|
}
|
|
|
|
fn geo_filtered_placeholder_search_should_not_return_deleted_documents_(
|
|
deletion_strategy: DeletionStrategy,
|
|
) {
|
|
let index = TempIndex::new();
|
|
|
|
let mut wtxn = index.write_txn().unwrap();
|
|
index
|
|
.update_settings_using_wtxn(&mut wtxn, |settings| {
|
|
settings.set_primary_key(S("id"));
|
|
settings.set_filterable_fields(hashset!(S("_geo")));
|
|
settings.set_sortable_fields(hashset!(S("_geo")));
|
|
})
|
|
.unwrap();
|
|
|
|
index.add_documents_using_wtxn(&mut wtxn, documents!([
|
|
{ "id": "1", "city": "Lille", "_geo": { "lat": 50.6299, "lng": 3.0569 } },
|
|
{ "id": "2", "city": "Mons-en-Barœul", "_geo": { "lat": 50.6415, "lng": 3.1106 } },
|
|
{ "id": "3", "city": "Hellemmes", "_geo": { "lat": 50.6312, "lng": 3.1106 } },
|
|
{ "id": "4", "city": "Villeneuve-d'Ascq", "_geo": { "lat": 50.6224, "lng": 3.1476 } },
|
|
{ "id": "5", "city": "Hem", "_geo": { "lat": 50.6552, "lng": 3.1897 } },
|
|
{ "id": "6", "city": "Roubaix", "_geo": { "lat": 50.6924, "lng": 3.1763 } },
|
|
{ "id": "7", "city": "Tourcoing", "_geo": { "lat": 50.7263, "lng": 3.1541 } },
|
|
{ "id": "8", "city": "Mouscron", "_geo": { "lat": 50.7453, "lng": 3.2206 } },
|
|
{ "id": "9", "city": "Tournai", "_geo": { "lat": 50.6053, "lng": 3.3758 } },
|
|
{ "id": "10", "city": "Ghent", "_geo": { "lat": 51.0537, "lng": 3.6957 } },
|
|
{ "id": "11", "city": "Brussels", "_geo": { "lat": 50.8466, "lng": 4.3370 } },
|
|
{ "id": "12", "city": "Charleroi", "_geo": { "lat": 50.4095, "lng": 4.4347 } },
|
|
{ "id": "13", "city": "Mons", "_geo": { "lat": 50.4502, "lng": 3.9623 } },
|
|
{ "id": "14", "city": "Valenciennes", "_geo": { "lat": 50.3518, "lng": 3.5326 } },
|
|
{ "id": "15", "city": "Arras", "_geo": { "lat": 50.2844, "lng": 2.7637 } },
|
|
{ "id": "16", "city": "Cambrai", "_geo": { "lat": 50.1793, "lng": 3.2189 } },
|
|
{ "id": "17", "city": "Bapaume", "_geo": { "lat": 50.1112, "lng": 2.8547 } },
|
|
{ "id": "18", "city": "Amiens", "_geo": { "lat": 49.9314, "lng": 2.2710 } },
|
|
{ "id": "19", "city": "Compiègne", "_geo": { "lat": 49.4449, "lng": 2.7913 } },
|
|
{ "id": "20", "city": "Paris", "_geo": { "lat": 48.9021, "lng": 2.3708 } }
|
|
])).unwrap();
|
|
|
|
let external_ids_to_delete = ["5", "6", "7", "12", "17", "19"];
|
|
let deleted_internal_ids =
|
|
delete_documents(&mut wtxn, &index, &external_ids_to_delete, deletion_strategy);
|
|
|
|
// Placeholder search with geo filter
|
|
let filter = Filter::from_str("_geoRadius(50.6924, 3.1763, 20000)").unwrap().unwrap();
|
|
let results = index.search(&wtxn).filter(filter).execute().unwrap();
|
|
assert!(!results.documents_ids.is_empty());
|
|
for id in results.documents_ids.iter() {
|
|
assert!(
|
|
!deleted_internal_ids.contains(id),
|
|
"The document {} was supposed to be deleted",
|
|
id
|
|
);
|
|
}
|
|
|
|
wtxn.commit().unwrap();
|
|
|
|
db_snap!(index, soft_deleted_documents_ids, deletion_strategy);
|
|
db_snap!(index, facet_id_f64_docids, deletion_strategy);
|
|
db_snap!(index, facet_id_string_docids, deletion_strategy);
|
|
}
|
|
|
|
#[test]
|
|
fn geo_filtered_placeholder_search_should_not_return_deleted_documents() {
|
|
geo_filtered_placeholder_search_should_not_return_deleted_documents_(
|
|
DeletionStrategy::AlwaysHard,
|
|
);
|
|
geo_filtered_placeholder_search_should_not_return_deleted_documents_(
|
|
DeletionStrategy::AlwaysSoft,
|
|
);
|
|
}
|
|
|
|
fn get_documents_should_not_return_deleted_documents_(deletion_strategy: DeletionStrategy) {
|
|
let index = TempIndex::new();
|
|
|
|
let mut wtxn = index.write_txn().unwrap();
|
|
index
|
|
.update_settings_using_wtxn(&mut wtxn, |settings| {
|
|
settings.set_primary_key(S("docid"));
|
|
})
|
|
.unwrap();
|
|
|
|
index
|
|
.add_documents_using_wtxn(
|
|
&mut wtxn,
|
|
documents!([
|
|
{ "docid": "1_4", "label": ["sign"] },
|
|
{ "docid": "1_5", "label": ["letter"] },
|
|
{ "docid": "1_7", "label": ["abstract","cartoon","design","pattern"] },
|
|
{ "docid": "1_36", "label": ["drawing","painting","pattern"] },
|
|
{ "docid": "1_37", "label": ["art","drawing","outdoor"] },
|
|
{ "docid": "1_38", "label": ["aquarium","art","drawing"] },
|
|
{ "docid": "1_39", "label": ["abstract"] },
|
|
{ "docid": "1_40", "label": ["cartoon"] },
|
|
{ "docid": "1_41", "label": ["art","drawing"] },
|
|
{ "docid": "1_42", "label": ["art","pattern"] },
|
|
{ "docid": "1_43", "label": ["abstract","art","drawing","pattern"] },
|
|
{ "docid": "1_44", "label": ["drawing"] },
|
|
{ "docid": "1_45", "label": ["art"] },
|
|
{ "docid": "1_46", "label": ["abstract","colorfulness","pattern"] },
|
|
{ "docid": "1_47", "label": ["abstract","pattern"] },
|
|
{ "docid": "1_52", "label": ["abstract","cartoon"] },
|
|
{ "docid": "1_57", "label": ["abstract","drawing","pattern"] },
|
|
{ "docid": "1_58", "label": ["abstract","art","cartoon"] },
|
|
{ "docid": "1_68", "label": ["design"] },
|
|
{ "docid": "1_69", "label": ["geometry"] },
|
|
{ "docid": "1_70", "label2": ["geometry", 1.2] },
|
|
{ "docid": "1_71", "label2": ["design", 2.2] },
|
|
{ "docid": "1_72", "label2": ["geometry", 1.2] }
|
|
]),
|
|
)
|
|
.unwrap();
|
|
|
|
let deleted_external_ids = ["1_7", "1_52"];
|
|
let deleted_internal_ids =
|
|
delete_documents(&mut wtxn, &index, &deleted_external_ids, deletion_strategy);
|
|
|
|
// list all documents
|
|
let results = index.all_documents(&wtxn).unwrap();
|
|
for result in results {
|
|
let (id, _) = result.unwrap();
|
|
assert!(
|
|
!deleted_internal_ids.contains(&id),
|
|
"The document {} was supposed to be deleted",
|
|
id
|
|
);
|
|
}
|
|
|
|
// list internal document ids
|
|
let results = index.documents_ids(&wtxn).unwrap();
|
|
for id in results {
|
|
assert!(
|
|
!deleted_internal_ids.contains(&id),
|
|
"The document {} was supposed to be deleted",
|
|
id
|
|
);
|
|
}
|
|
wtxn.commit().unwrap();
|
|
|
|
let rtxn = index.read_txn().unwrap();
|
|
|
|
// get internal docids from deleted external document ids
|
|
let results = index.external_documents_ids(&rtxn).unwrap();
|
|
for id in deleted_external_ids {
|
|
assert!(results.get(id).is_none(), "The document {} was supposed to be deleted", id);
|
|
}
|
|
drop(rtxn);
|
|
|
|
db_snap!(index, soft_deleted_documents_ids, deletion_strategy);
|
|
}
|
|
|
|
#[test]
|
|
fn get_documents_should_not_return_deleted_documents() {
|
|
get_documents_should_not_return_deleted_documents_(DeletionStrategy::AlwaysHard);
|
|
get_documents_should_not_return_deleted_documents_(DeletionStrategy::AlwaysSoft);
|
|
}
|
|
|
|
fn stats_should_not_return_deleted_documents_(deletion_strategy: DeletionStrategy) {
|
|
let index = TempIndex::new();
|
|
|
|
let mut wtxn = index.write_txn().unwrap();
|
|
|
|
index
|
|
.update_settings_using_wtxn(&mut wtxn, |settings| {
|
|
settings.set_primary_key(S("docid"));
|
|
})
|
|
.unwrap();
|
|
|
|
index.add_documents_using_wtxn(&mut wtxn, documents!([
|
|
{ "docid": "1_4", "label": ["sign"]},
|
|
{ "docid": "1_5", "label": ["letter"]},
|
|
{ "docid": "1_7", "label": ["abstract","cartoon","design","pattern"], "title": "Mickey Mouse"},
|
|
{ "docid": "1_36", "label": ["drawing","painting","pattern"]},
|
|
{ "docid": "1_37", "label": ["art","drawing","outdoor"]},
|
|
{ "docid": "1_38", "label": ["aquarium","art","drawing"], "title": "Nemo"},
|
|
{ "docid": "1_39", "label": ["abstract"]},
|
|
{ "docid": "1_40", "label": ["cartoon"]},
|
|
{ "docid": "1_41", "label": ["art","drawing"]},
|
|
{ "docid": "1_42", "label": ["art","pattern"]},
|
|
{ "docid": "1_43", "label": ["abstract","art","drawing","pattern"], "number": 32i32},
|
|
{ "docid": "1_44", "label": ["drawing"], "number": 44i32},
|
|
{ "docid": "1_45", "label": ["art"]},
|
|
{ "docid": "1_46", "label": ["abstract","colorfulness","pattern"]},
|
|
{ "docid": "1_47", "label": ["abstract","pattern"]},
|
|
{ "docid": "1_52", "label": ["abstract","cartoon"]},
|
|
{ "docid": "1_57", "label": ["abstract","drawing","pattern"]},
|
|
{ "docid": "1_58", "label": ["abstract","art","cartoon"]},
|
|
{ "docid": "1_68", "label": ["design"]},
|
|
{ "docid": "1_69", "label": ["geometry"]}
|
|
])).unwrap();
|
|
|
|
delete_documents(&mut wtxn, &index, &["1_7", "1_52"], deletion_strategy);
|
|
|
|
// count internal documents
|
|
let results = index.number_of_documents(&wtxn).unwrap();
|
|
assert_eq!(18, results);
|
|
|
|
// count field distribution
|
|
let results = index.field_distribution(&wtxn).unwrap();
|
|
assert_eq!(Some(&18), results.get("label"));
|
|
assert_eq!(Some(&1), results.get("title"));
|
|
assert_eq!(Some(&2), results.get("number"));
|
|
|
|
wtxn.commit().unwrap();
|
|
|
|
db_snap!(index, soft_deleted_documents_ids, deletion_strategy);
|
|
}
|
|
|
|
#[test]
|
|
fn stats_should_not_return_deleted_documents() {
|
|
stats_should_not_return_deleted_documents_(DeletionStrategy::AlwaysHard);
|
|
stats_should_not_return_deleted_documents_(DeletionStrategy::AlwaysSoft);
|
|
}
|
|
}
|