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

Standard view columns interact force builds to be eagerly loaded

    XMLWordPrintable

Details

    Description

      LastFailureColumn, shown by default in AllView, is calling Job.getLastFailedBuild which then keeps on loading builds until it finds a failed build, with no cutoff. The fix for that would be to either impose a maximum number of builds to search back from (10? 20?); or do the search asynchronously after the page loads, which might still put some load on the server but would at least not stop you from browsing.

      The same would apply to the last success column if all your recent builds failed, etc.

      Attachments

        Issue Links

          depends on

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-18846 New lazy loading permalinks can break job.lastStableBuild != null => job.lastSuccessfulBuild != null

          • Major - Major loss of function.
          • Resolved
          is blocking

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-16023 Lazy loading causes massive delays after a period of inactivity when loading dashboard

          • Major - Major loss of function.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. JENKINS-8754 ROADMAP: Improve Start-up Time

          • Major - Major loss of function.
          • Closed
          is related to

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-17449 Jenkins 1.509 has different behaviour with symlink handling of e.g. lastSuccessful

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Activity

            Ascending order - Click to sort in descending order
            View 7 older comments
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Kohsuke Kawaguchi
            Path:
            core/src/main/java/hudson/model/AbstractBuild.java
            test/src/test/java/hudson/model/AbstractProjectTest.java
            http://jenkins-ci.org/commit/jenkins/4d0def46ccf9e97074b3f4b4e66ebfe9ead9b74b
            Log:
            JENKINS-16089 needs a separator here

            And since it's now a symlink that points to another symlink that then
            points to the real stuff, we need recursive onion peeling.

            Compare: https://github.com/jenkinsci/jenkins/compare/2aa0293858d5...4d0def46ccf9

            –
            You received this message because you are subscribed to the Google Groups "Jenkins Commits" group.
            To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-commits+unsubscribe@googlegroups.com.
            For more options, visit https://groups.google.com/groups/opt_out.

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Kohsuke Kawaguchi Path: core/src/main/java/hudson/model/AbstractBuild.java test/src/test/java/hudson/model/AbstractProjectTest.java http://jenkins-ci.org/commit/jenkins/4d0def46ccf9e97074b3f4b4e66ebfe9ead9b74b Log: JENKINS-16089 needs a separator here And since it's now a symlink that points to another symlink that then points to the real stuff, we need recursive onion peeling. Compare: https://github.com/jenkinsci/jenkins/compare/2aa0293858d5...4d0def46ccf9 – You received this message because you are subscribed to the Google Groups "Jenkins Commits" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-commits+unsubscribe@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out .
            dogfood dogfood added a comment -

            Integrated in jenkins_main_trunk #2370
            JENKINS-16089 needs a separator here (Revision 4d0def46ccf9e97074b3f4b4e66ebfe9ead9b74b)

            Result = SUCCESS
            kohsuke : 4d0def46ccf9e97074b3f4b4e66ebfe9ead9b74b
            Files :

            • core/src/main/java/hudson/model/AbstractBuild.java
            • test/src/test/java/hudson/model/AbstractProjectTest.java
            dogfood dogfood added a comment - Integrated in jenkins_main_trunk #2370 JENKINS-16089 needs a separator here (Revision 4d0def46ccf9e97074b3f4b4e66ebfe9ead9b74b) Result = SUCCESS kohsuke : 4d0def46ccf9e97074b3f4b4e66ebfe9ead9b74b Files : core/src/main/java/hudson/model/AbstractBuild.java test/src/test/java/hudson/model/AbstractProjectTest.java
            martinkutter Martin Kutter added a comment -

            Added lts-candidate label, as this item mainly affects bigger installations (with many jobs in a view)

            martinkutter Martin Kutter added a comment - Added lts-candidate label, as this item mainly affects bigger installations (with many jobs in a view)
            jglick Jesse Glick added a comment -

            @martinkutter the fix was already put in 1.507 so lts-candidate is irrelevant now.

            jglick Jesse Glick added a comment - @martinkutter the fix was already put in 1.507 so lts-candidate is irrelevant now.
            martinkutter Martin Kutter added a comment -

            @JesseGlick: Sorry, you're right - label removed

            martinkutter Martin Kutter added a comment - @JesseGlick: Sorry, you're right - label removed

            People

              justinsb Justin Santa Barbara
              jglick Jesse Glick
              Votes:
              5 Vote for this issue
              Watchers:
              12 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: