This PR improves the performance of the MRCM type ahead search widgets.
It addresses the following issues:
using filterByExtendedLocales(...) is expensive, it is safe to use only setLocales(...) in this context. Expansion will use the locales, the results of the filterByTerm(...) are not affected by the change.
Inactivation property expansion is another expensive join operation, which is not really required when filtering by term in a concept search request, therefore I've added the expansion property inactivationProperties to the SnomedDescriptionConverter so the client has to explicitly require them.
If any client of the REST API is using either the inactivationIndicator or associationTargets, then please make sure you request them by using expand=inactivationProperties query parameter.
This PR improves the performance of the MRCM type ahead search widgets. It addresses the following issues:
filterByExtendedLocales(...)
is expensive, it is safe to use onlysetLocales(...)
in this context. Expansion will use the locales, the results of thefilterByTerm(...)
are not affected by the change.inactivationProperties
to theSnomedDescriptionConverter
so the client has to explicitly require them.If any client of the REST API is using either the
inactivationIndicator
orassociationTargets
, then please make sure you request them by usingexpand=inactivationProperties
query parameter.