snowykami/meilisearch
1
0
Fork 0
mirror of https://github.com/meilisearch/meilisearch.git synced 2024-11-30 09:04:59 +08:00
Code Issues Actions 3 Packages Projects Releases Wiki Activity

add primary key to fields_id_map when not present

Browse Source
This commit is contained in:
mpostma 2021-02-19 09:54:31 +01:00
parent 8dcb3e0c41
commit e08b6b3ec7
No known key found for this signature in database
GPG Key ID: CBC8A7C1D7A28C3A
1 changed files with 1 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
milli/src/update/index_documents/transform.rs
View File

@ -516,7 +516,7 @@ fn compute_primary_key_pair(
) -> anyhow::Result<(FieldId, String)> { ) -> anyhow::Result<(FieldId, String)> {
match primary_key { match primary_key {
Some(primary_key) => { Some(primary_key) => {
let id = fields_ids_map.id(primary_key).expect("primary key must be present in the fields id map"); let id = fields_ids_map.insert(primary_key).ok_or(anyhow!("Maximum number of fields exceeded"))?;
Ok((id, primary_key.to_string())) Ok((id, primary_key.to_string()))
} }
None => { None => {
@ -572,17 +572,6 @@ mod test {
use super::compute_primary_key_pair; use super::compute_primary_key_pair;
use super::FieldsIdsMap; use super::FieldsIdsMap;
#[test]
#[should_panic]
fn should_panic_primary_key_not_in_map() {
let mut fields_map = FieldsIdsMap::new();
let _result = compute_primary_key_pair(
Some("toto"),
&mut fields_map,
None,
false);
}
#[test] #[test]
fn should_return_primary_key_if_is_some() { fn should_return_primary_key_if_is_some() {
let mut fields_map = FieldsIdsMap::new(); let mut fields_map = FieldsIdsMap::new();