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

Standard view columns interact force builds to be eagerly loaded

    Details

    • Similar Issues:

      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

            Activity

            Hide
            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.

            Show
            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 .
            Hide
            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
            Show
            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
            Hide
            martinkutter Martin Kutter added a comment -

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

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

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

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

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

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

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: