Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-49086

Lazy-initialize FlowNodeStorage and heads, etc

    Details

    • Similar Issues:

      Description

      When loading Pipeline builds (WorkflowRuns) they eagerly load the FlowHeads, etc as part of deserialization.  This triggers additional disk hits and initialization to create the FlowNodeStorage and populate it with those nodes. 

      We should lazy-load these for completed builds and avoid fetching heads until requested – this makes operations that iterate through builds much faster. 

      This is especially important because the Bulk FlowNode storage (https://issues.jenkins-ci.org/browse/JENKINS-47173) will load all the nodes when created – which will happen when loading the build.  The CPU and disk hit for deserialization may be rather significant.   

        Attachments

          Issue Links

            Activity

            svanoort Sam Van Oort created issue -
            svanoort Sam Van Oort made changes -
            Field Original Value New Value
            Epic Link JENKINS-47170 [ 185575 ]
            svanoort Sam Van Oort made changes -
            Link This issue is related to JENKINS-39489 [ JENKINS-39489 ]
            svanoort Sam Van Oort made changes -
            Labels performance pipeline performance pipeline project-cheetah
            jglick Jesse Glick made changes -
            Link This issue relates to JENKINS-47173 [ JENKINS-47173 ]
            jglick Jesse Glick made changes -
            Link This issue relates to JENKINS-45585 [ JENKINS-45585 ]
            svanoort Sam Van Oort made changes -
            Link This issue duplicates JENKINS-45585 [ JENKINS-45585 ]
            svanoort Sam Van Oort made changes -
            Status Open [ 1 ] Closed [ 6 ]
            Resolution Duplicate [ 3 ]
            svanoort Sam Van Oort made changes -
            Component/s workflow-job-plugin [ 21716 ]
            Component/s workflow-cps-plugin [ 21713 ]

              People

              • Assignee:
                svanoort Sam Van Oort
                Reporter:
                svanoort Sam Van Oort
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: