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

Major heap consumption from merge MercurialChangeSet paths

    Details

    • Similar Issues:

      Description

      A heap dump of Hudson after OOME showed a lot of big strings coming from MercurialChangeSet in the affectedPaths, added, modified, and deleted fields. The twist is that these were from merge changesets, for which no paths are displayed in the GUI anyway (since they are generally meaningless).

      Paths should never be stored for a merge changeset to begin with. (Then the Jelly script also need not handle this specially.)

        Attachments

          Activity

          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in hudson
          User: : jglick
          Path:
          trunk/hudson/plugins/mercurial/src/main/java/hudson/plugins/mercurial/MercurialChangeSet.java
          trunk/hudson/plugins/mercurial/src/main/resources/hudson/plugins/mercurial/MercurialChangeSetList/index.jelly
          trunk/hudson/plugins/mercurial/src/test/java/hudson/plugins/mercurial/MercurialSCMTest.java
          http://jenkins-ci.org/commit/30981
          Log:
          [FIXED JENKINS-6517] Major heap consumption from merge MercurialChangeSet paths.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : jglick Path: trunk/hudson/plugins/mercurial/src/main/java/hudson/plugins/mercurial/MercurialChangeSet.java trunk/hudson/plugins/mercurial/src/main/resources/hudson/plugins/mercurial/MercurialChangeSetList/index.jelly trunk/hudson/plugins/mercurial/src/test/java/hudson/plugins/mercurial/MercurialSCMTest.java http://jenkins-ci.org/commit/30981 Log: [FIXED JENKINS-6517] Major heap consumption from merge MercurialChangeSet paths.

            People

            • Assignee:
              jglick Jesse Glick
              Reporter:
              jglick Jesse Glick
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: