Uploaded image for project: 'Magnolia REST Framework'
  1. Magnolia REST Framework
  2. MGNLREST-800

Bad performance on resolver resolution

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Neutral Neutral
    • 2.2.27, 3.0.0
    • 2.2.26
    • delivery
    • Yes

      Steps to reproduce

      1. Uses nightly-webapp
      2. Add some log in JcrDeliveryEndpoint.instantiatePredicate
      3. Access <magnolia host>/.rest/delivery/tours/v1

      Expected results

      getParent is called only when neccessary

      Actual results

      getParent is called with each reference node's properties

      Workaround

      Development notes

      • With original tour_v1.yaml
        Original code: getParent is called ~1026
        Patch code: getParent is not called
        New PR code: getParent is not called
      • Modify tour_v1.yaml to simulate nodeType!=null by adding nodeType: node
        Original code: getParent is called ~1026
        Patch code: getParent is called ~1026
        New PR code: getParent is called ~109

        Acceptance criteria

              plequoc Phong Le Quoc
              frank.sommer Frank Sommer
              DeveloperX
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved:
                Work Started:

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - Not Specified
                  Not Specified
                  Logged:
                  Time Spent - 7d 1.5h
                  7d 1.5h