Uploaded image for project: 'OpenIDM'
  1. OpenIDM
  2. OPENIDM-10504

Refactor of mutable map in GraphQueryHandler#executeQueryStatement

    Details

      Description

      Based on feedback in this PR:

      https://stash.forgerock.org/projects/OPENIDM/repos/openidm/pull-requests/3750/overview?commentId=350393

       

      Instead of Loading the relationship map in a forEach loop, create the map by using a collector.

      Then rather than clearing the map when moving to the next object to be handled, reset the   RelationshipMergeSets.  This will eliminate the any mutation in the map keys or value references.

      `final Map<String, RelationshipMergeSet> relationships = new HashMap<>();`

       

        Attachments

          Activity

            People

            • Assignee:
              jason.vincent jason vincent
              Reporter:
              jason.vincent jason vincent
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: