hapifhir / hapi-fhir

🔥 HAPI FHIR - Java API for HL7 FHIR Clients and Servers
http://hapifhir.io
Apache License 2.0
1.94k stars 1.3k forks source link

Search Param Matcher Performance Improvement #5999

Closed melihaydogd closed 2 weeks ago

melihaydogd commented 2 weeks ago

extractIndexedSearchParams function uses ISearchParamExtractor.ALL_PARAMS by default. This tries to extract all possible values from a resource then tries to match the resource. By adding the getFilter function, we can extract only the values that is in search parameter map. I saw that this had a good impact on the performance of search param matcher.

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.43%. Comparing base (497b9f2) to head (7946ff1). Report is 93 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #5999 +/- ## ============================================ + Coverage 83.39% 83.43% +0.03% - Complexity 26927 27228 +301 ============================================ Files 1681 1695 +14 Lines 103965 105333 +1368 Branches 13189 13317 +128 ============================================ + Hits 86702 87880 +1178 - Misses 11613 11747 +134 - Partials 5650 5706 +56 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.