Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Commit

Permalink
Rename the FieldIdMapMissingEntry from_db_name field into process
Browse files Browse the repository at this point in the history
  • Loading branch information
Kerollmops committed Jun 15, 2021
1 parent 28c004a commit f0e804a
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 9 deletions.
12 changes: 6 additions & 6 deletions milli/src/error.rs
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,8 @@ pub enum SerializationError {

#[derive(Debug)]
pub enum FieldIdMapMissingEntry {
FieldId { field_id: FieldId, from_db_name: &'static str },
FieldName { field_name: String, from_db_name: &'static str },
FieldId { field_id: FieldId, process: &'static str },
FieldName { field_name: String, process: &'static str },
}

#[derive(Debug)]
Expand Down Expand Up @@ -224,11 +224,11 @@ impl StdError for UserError {}
impl fmt::Display for FieldIdMapMissingEntry {
fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
match self {
Self::FieldId { field_id, from_db_name } => {
write!(f, "unknown field id {} coming from {} database", field_id, from_db_name)
Self::FieldId { field_id, process } => {
write!(f, "unknown field id {} coming from the {} process", field_id, process)
},
Self::FieldName { field_name, from_db_name } => {
write!(f, "unknown field name {} coming from {} database", field_name, from_db_name)
Self::FieldName { field_name, process } => {
write!(f, "unknown field name {} coming from the {} process", field_name, process)
},
}
}
Expand Down
2 changes: 1 addition & 1 deletion milli/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ pub fn obkv_to_json(
.map(|(id, value)| {
let name = fields_ids_map.name(id).ok_or(error::FieldIdMapMissingEntry::FieldId {
field_id: id,
from_db_name: "documents",
process: "obkv_to_json",
})?;
let value = serde_json::from_slice(value).map_err(error::InternalError::SerdeJson)?;
Ok((name.to_owned(), value))
Expand Down
2 changes: 1 addition & 1 deletion milli/src/search/criteria/asc_desc.rs
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ impl<'t> AscDesc<'t> {
.id(&field_name)
.ok_or_else(|| FieldIdMapMissingEntry::FieldName {
field_name: field_name.clone(),
from_db_name: "asc-desc",
process: "AscDesc::new",
})?;

Ok(AscDesc {
Expand Down
2 changes: 1 addition & 1 deletion milli/src/search/facet/facet_distribution.rs
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,7 @@ impl<'a> FacetDistribution<'a> {
for name in filterable_fields {
let fid = fields_ids_map.id(&name).ok_or_else(|| FieldIdMapMissingEntry::FieldName {
field_name: name.clone(),
from_db_name: "filterable-fields",
process: "FacetDistribution::execute",
})?;
let values = self.facet_values(fid)?;
distribution.insert(name, values);
Expand Down

0 comments on commit f0e804a

Please sign in to comment.