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

Improve cobertura plugin memory usage when drawing trend graphs

    XMLWordPrintable

    Details

    • Type: Patch
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Won't Fix
    • Component/s: cobertura-plugin
    • Labels:
      None
    • Environment:
      Platform: All, OS: All
    • Similar Issues:

      Description

      The majority of the memory usage of the Cobertura plugin is currently in the
      "paint" object. However, the "paint" object is only used during initial
      publishing, not during subsequent trend change generation or metrics display.

      This patch pulls the "paint" support out of CoverageResult into a subclass,
      PaintedCoverageResult. The parser is modified to return the basic
      CoverageResult by default, and adds methods to request a PaintedCoverageResult
      (which are used by the publishers).

      After this patch, the Cobertura plugin will still be slow when large coverage
      files and/or lots of builds are in play, but it should throw OutOfMemoryErrors
      on trend graph generation less commonly.

        Attachments

          Issue Links

            Activity

            davidmc24 davidmc24 created issue -
            davidmc24 davidmc24 made changes -
            Field Original Value New Value
            Status Open [ 1 ] In Progress [ 3 ]
            scm_issue_link SCM/JIRA link daemon made changes -
            Status In Progress [ 3 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]
            davidmc24 davidmc24 made changes -
            Status Resolved [ 5 ] Closed [ 6 ]
            davidmc24 davidmc24 made changes -
            Resolution Fixed [ 1 ]
            Status Closed [ 6 ] Reopened [ 4 ]
            mdonohue mdonohue made changes -
            Link This issue is duplicated by JENKINS-4795 [ JENKINS-4795 ]
            davidmc24 davidmc24 made changes -
            Status Reopened [ 4 ] Closed [ 6 ]
            Resolution Won't Fix [ 2 ]
            rtyler R. Tyler Croy made changes -
            Workflow JNJira [ 133670 ] JNJira + In-Review [ 202333 ]

              People

              • Assignee:
                davidmc24 davidmc24
                Reporter:
                davidmc24 davidmc24
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: