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

WorkflowRun.onLoad need not eagerly load the FlowExecution of a completed build

    Details

    • Similar Issues:

      Description

      Currently we always load FlowExecution when loading a build even though we might not actually need it. This is thought to slow down some things, such as getBuildHealth, since we are loading executions for some historical completed builds and then not using them. The loading should be on demand only.

      Since currently FlowExecution.onComplete is called soon thereafter, we would need some other marker in WorkflowRun for a completed build. Could check for logsToCopy == null though this is deleted in JENKINS-38381. Probably better to use completed as noted here.

        Attachments

          Issue Links

            Activity

            jglick Jesse Glick created issue -
            cloudbees CloudBees Inc. made changes -
            Field Original Value New Value
            Remote Link This issue links to "CloudBees Internal CD-20 (Web Link)" [ 19118 ]
            jglick Jesse Glick made changes -
            Link This issue relates to JENKINS-50030 [ JENKINS-50030 ]
            jglick Jesse Glick made changes -
            Link This issue relates to JENKINS-49086 [ JENKINS-49086 ]
            svanoort Sam Van Oort made changes -
            Assignee Sam Van Oort [ svanoort ]
            svanoort Sam Van Oort made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            svanoort Sam Van Oort made changes -
            Status In Progress [ 3 ] In Review [ 10005 ]
            svanoort Sam Van Oort made changes -
            Link This issue is duplicated by JENKINS-49086 [ JENKINS-49086 ]
            svanoort Sam Van Oort made changes -
            Status In Review [ 10005 ] Closed [ 6 ]
            Resolution Fixed [ 1 ]
            svanoort Sam Van Oort made changes -
            Link This issue is related to JENKINS-47224 [ JENKINS-47224 ]
            svanoort Sam Van Oort made changes -
            Link This issue is duplicated by JENKINS-47224 [ JENKINS-47224 ]

              People

              • Assignee:
                svanoort Sam Van Oort
                Reporter:
                jglick Jesse Glick
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: