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

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

XMLWordPrintable

      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.

          [JENKINS-45585] WorkflowRun.onLoad need not eagerly load the FlowExecution of a completed build

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

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

              Created:
              Updated:
              Resolved: