You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
After #4833 lands, there will be a degradation in performance of primary-key queries compared to before (linear scan is being used instead of looking up a map). An special case should be created for the primary-key queries, as outlined in #4833 (comment):
let iter:Box<dynIterator<Item = Domain>> = match filter {CompoundPredicate::Atom(DomainPredicateBox::Id(DomainIdPredicateBox::Equals(
domain_id,))) => Box::new(Some(state_ro
.world().domain(&domain_id)?
.clone()).into_iter()),
filter => Box::new(
state_ro
.world().domains_iter().filter(move |&v| filter.applies(v)).cloned(),),};Ok(iter)
The text was updated successfully, but these errors were encountered:
TL;DR
After #4833 lands, there will be a degradation in performance of primary-key queries compared to before (linear scan is being used instead of looking up a map). An special case should be created for the primary-key queries, as outlined in #4833 (comment):
The text was updated successfully, but these errors were encountered: