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

JDepend Big config file with enabled plugin

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: plugin-proposals
    • Labels:
      None
    • Environment:
      Platform: All, OS: All
    • Similar Issues:

      Description

      Hudson 1.323
      JDepend-Plugin: 1.2.1

      I just observed, that enabling jdepend let the config.xml file of a job grow from 2kB to 56kB. In the diff
      I see a big serialized blob for the PrintStream logger in hudson.plugins.jdepend.JDependRecorder.

      Looking at:
      https://svn.dev.java.net/svn/hudson/trunk/hudson/plugins/jdepend/src/main/java/hudson/plugins/j
      depend/JDependRecorder.java

      Propably the field logger should be transient, so it is not serialized to disk.

      Regards
      Mirko

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          lewisham Lewisham added a comment -

          Learn something new every day: having never had to serialize an object, I never
          learnt about the transient keyword

          JDepend 1.2.2 has been pushed to the repo with your proposed fix. Feel free to
          reopen this ticket if you get any unexpected behavior.

          Show
          lewisham Lewisham added a comment - Learn something new every day: having never had to serialize an object, I never learnt about the transient keyword JDepend 1.2.2 has been pushed to the repo with your proposed fix. Feel free to reopen this ticket if you get any unexpected behavior.
          Hide
          Permalink
          mfriedenhagen Mirko Friedenhagen added a comment -

          The error is gone.

          Show
          mfriedenhagen Mirko Friedenhagen added a comment - The error is gone.

            People

            Assignee:
            lewisham Lewisham
            Reporter:
            mfriedenhagen Mirko Friedenhagen
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: