Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

FML transform performance tuning #1704

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

FML transform performance tuning #1704

wants to merge 5 commits into from
+77 −21

Commits on Nov 2, 2024

  1. Perf tuning: skip numberChildren 

    hapifhir#1699 Workaround for performance issue with FML transform: Growing/large lists getting very slow due to numberChildren() getting called recursively for every element that is added to a list.

merge upstream
    @mrunibe
    mrunibe committed Nov 2, 2024
    Configuration menu
    Copy the full SHA
    18b4fc1 View commit details
    Browse the repository at this point in the history

  2. Perf tuning FML: re-use FHIRPathEngine in new Property 

    hapifhir#1703 FML transform create heavyweight
FHIRPathEngine objects for every new Property. Re-use existing
ProfileUtilities/FHIRPathEngine objects from StructureMapUtilities.
    @mrunibe
    mrunibe committed Nov 2, 2024
    Configuration menu
    Copy the full SHA
    b75e401 View commit details
    Browse the repository at this point in the history

  3. Perf tuning: skip numberChildren 

    hapifhir#1699 defer numberChildren to end of transform
    @mrunibe
    mrunibe committed Nov 2, 2024
    Configuration menu
    Copy the full SHA
    d90c6b3 View commit details
    Browse the repository at this point in the history

  4. hapifhir#1699 Testcase for numberChildren

    @mrunibe
    mrunibe committed Nov 2, 2024
    Configuration menu
    Copy the full SHA
    f909a0c View commit details
    Browse the repository at this point in the history

  5. hapifhir#1699 set index when adding children 

    instead of calling numberChildren() recursively whenever an element is
added.
    @mrunibe
    mrunibe committed Nov 2, 2024
    Configuration menu
    Copy the full SHA
    8d780e0 View commit details
    Browse the repository at this point in the history