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

Fix bug in string facet distribution with few candidates

Browse Source
This commit is contained in:
Loïc Lecrenier 2022-09-07 18:00:00 +02:00 committed by Loïc Lecrenier
parent fca4577e23
commit 3d7ed3263f
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
milli/src/search/facet/facet_distribution.rs
View File

@ -96,7 +96,7 @@ impl<'a> FacetDistribution<'a> {
let mut key_buffer: Vec<_> = field_id.to_be_bytes().iter().copied().collect(); let mut key_buffer: Vec<_> = field_id.to_be_bytes().iter().copied().collect();
let db = self.index.field_id_docid_facet_strings; let db = self.index.field_id_docid_facet_strings;
for docid in candidates.into_iter() { 'outer: for docid in candidates.into_iter() {
key_buffer.truncate(mem::size_of::<FieldId>()); key_buffer.truncate(mem::size_of::<FieldId>());
key_buffer.extend_from_slice(&docid.to_be_bytes()); key_buffer.extend_from_slice(&docid.to_be_bytes());
let iter = db let iter = db
@ -112,7 +112,7 @@ impl<'a> FacetDistribution<'a> {
*count += 1; *count += 1;
if normalized_distribution.len() == self.max_values_per_facet { if normalized_distribution.len() == self.max_values_per_facet {
break; break 'outer;
} }
} }
} }
@ -393,7 +393,7 @@ mod tests {
.execute() .execute()
.unwrap(); .unwrap();
milli_snap!(format!("{map:?}"), @r###"{"colour": {"Blue": 2, "RED": 1}}"###); milli_snap!(format!("{map:?}"), @r###"{"colour": {"Blue": 1}}"###);
} }
#[test] #[test]