Rename editionCode to function

This commit is contained in:
Clément Renault 2024-05-10 19:42:44 +02:00
parent f4add93043
commit f32e6c32fc
No known key found for this signature in database
GPG Key ID: F250A4C4E3AE5F5F
5 changed files with 22 additions and 23 deletions

View File

@ -1411,28 +1411,27 @@ impl IndexScheduler {
Ok(tasks)
}
IndexOperation::DocumentEdition { mut task, .. } => {
let (filter, edition_code) =
if let KindWithContent::DocumentEdition { filter_expr, edition_code, .. } =
let (filter, function) =
if let KindWithContent::DocumentEdition { filter_expr, function, .. } =
&task.kind
{
(filter_expr, edition_code)
(filter_expr, function)
} else {
unreachable!()
};
let edited_documents = edit_documents_by_function(
index_wtxn,
filter,
edition_code,
function,
self.index_mapper.indexer_config(),
self.must_stop_processing.clone(),
index,
);
let (original_filter, edition_code) =
if let Some(Details::DocumentEdition {
original_filter, edition_code, ..
}) = task.details
let (original_filter, function) =
if let Some(Details::DocumentEdition { original_filter, function, .. }) =
task.details
{
(original_filter, edition_code)
(original_filter, function)
} else {
// In the case of a `documentDeleteByFilter` the details MUST be set
unreachable!();
@ -1443,7 +1442,7 @@ impl IndexScheduler {
task.status = Status::Succeeded;
task.details = Some(Details::DocumentEdition {
original_filter,
edition_code,
function,
edited_documents: Some(edited_documents),
});
}
@ -1451,7 +1450,7 @@ impl IndexScheduler {
task.status = Status::Failed;
task.details = Some(Details::DocumentEdition {
original_filter,
edition_code,
function,
edited_documents: Some(0),
});
task.error = Some(e.into());

View File

@ -179,10 +179,10 @@ fn snapshot_details(d: &Details) -> String {
}
Details::DocumentEdition {
edited_documents,
edition_code,
function,
original_filter,
} => {
format!("{{ edited_documents: {edited_documents:?}, edition_code: {edition_code:?}, original_filter: {original_filter:?} }}")
format!("{{ edited_documents: {edited_documents:?}, function: {function:?}, original_filter: {original_filter:?} }}")
}
Details::SettingsUpdate { settings } => {
format!("{{ settings: {settings:?} }}")

View File

@ -72,7 +72,7 @@ pub struct DetailsView {
#[serde(skip_serializing_if = "Option::is_none")]
pub dump_uid: Option<Option<String>>,
#[serde(skip_serializing_if = "Option::is_none")]
pub edition_code: Option<String>,
pub function: Option<String>,
#[serde(skip_serializing_if = "Option::is_none")]
#[serde(flatten)]
pub settings: Option<Box<Settings<Unchecked>>>,
@ -90,11 +90,11 @@ impl From<Details> for DetailsView {
..DetailsView::default()
}
}
Details::DocumentEdition { edited_documents, original_filter, edition_code } => {
Details::DocumentEdition { edited_documents, original_filter, function } => {
DetailsView {
edited_documents: Some(edited_documents),
original_filter: Some(original_filter),
edition_code: Some(edition_code),
function: Some(function),
..DetailsView::default()
}
}

View File

@ -99,7 +99,7 @@ pub enum KindWithContent {
DocumentEdition {
index_uid: String,
filter_expr: Option<serde_json::Value>,
edition_code: String,
function: String,
},
DocumentDeletion {
index_uid: String,
@ -211,11 +211,11 @@ impl KindWithContent {
indexed_documents: None,
})
}
KindWithContent::DocumentEdition { index_uid: _, edition_code, filter_expr } => {
KindWithContent::DocumentEdition { index_uid: _, function, filter_expr } => {
Some(Details::DocumentEdition {
edited_documents: None,
original_filter: filter_expr.as_ref().map(|v| v.to_string()),
edition_code: edition_code.clone(),
function: function.clone(),
})
}
KindWithContent::DocumentDeletion { index_uid: _, documents_ids } => {
@ -266,11 +266,11 @@ impl KindWithContent {
indexed_documents: Some(0),
})
}
KindWithContent::DocumentEdition { index_uid: _, filter_expr, edition_code } => {
KindWithContent::DocumentEdition { index_uid: _, filter_expr, function } => {
Some(Details::DocumentEdition {
edited_documents: Some(0),
original_filter: filter_expr.as_ref().map(|v| v.to_string()),
edition_code: edition_code.clone(),
function: function.clone(),
})
}
KindWithContent::DocumentDeletion { index_uid: _, documents_ids } => {
@ -531,7 +531,7 @@ pub enum Details {
DocumentEdition {
edited_documents: Option<u64>,
original_filter: Option<String>,
edition_code: String,
function: String,
},
SettingsUpdate {
settings: Box<Settings<Unchecked>>,

View File

@ -613,7 +613,7 @@ pub async fn edit_documents_by_function(
.map_err(|err| ResponseError::from_msg(err.message, Code::InvalidDocumentFilter))?;
}
let task =
KindWithContent::DocumentEdition { index_uid, filter_expr: filter, edition_code: function };
KindWithContent::DocumentEdition { index_uid, filter_expr: filter, function: function };
let uid = get_task_id(&req, &opt)?;
let dry_run = is_dry_run(&req, &opt)?;