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

Excessive warnings on exception while evaluating exeparent.url

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • distfork-plugin
    • None

      I am using the distfork plugin intensively. In the Jenkins log file there are many warnings like this:

      Jun 25, 2014 1:36:12 AM hudson.ExpressionFactory2$JexlExpression evaluate
      WARNING: Caught exception evaluating: exeparent.url in /. Reason: java.lang.reflect.InvocationTargetException
      java.lang.reflect.InvocationTargetException
      at sun.reflect.GeneratedMethodAccessor336.invoke(Unknown Source)
      at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
      at java.lang.reflect.Method.invoke(Method.java:597)
      at org.apache.commons.jexl.util.PropertyExecutor.execute(PropertyExecutor.java:125)
      .....
      at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
      at java.lang.Thread.run(Thread.java:662)
      Caused by: java.lang.UnsupportedOperationException
      at hudson.plugins.distfork.DistForkTask.getUrl(DistForkTask.java:102)
      ... 191 more

      These are only warnings, but they cause huge jenkins log files.

      How can these warnings be avoided?

          [JENKINS-23567] Excessive warnings on exception while evaluating exeparent.url

          Daniel Beck added a comment -

          Caused by:
          https://github.com/jenkinsci/distfork-plugin/blob/master/src/main/java/hudson/plugins/distfork/DistForkTask.java#L104

          I wonder what happens when this returns null instead. Ideally Jenkins would just assume there's no URL for the task.

          Daniel Beck added a comment - Caused by: https://github.com/jenkinsci/distfork-plugin/blob/master/src/main/java/hudson/plugins/distfork/DistForkTask.java#L104 I wonder what happens when this returns null instead. Ideally Jenkins would just assume there's no URL for the task.

          Jan Hollevoet added a comment -

          I did it like this a while ago

          public String getUrl()

          { String url = ""; return url; }

          That removed the warnings (of course) and the plugin runs nicely. Didn't try with returning null (yet).

          Jan Hollevoet added a comment - I did it like this a while ago public String getUrl() { String url = ""; return url; } That removed the warnings (of course) and the plugin runs nicely. Didn't try with returning null (yet).

          Code changed in jenkins
          User: James Nord
          Path:
          src/main/java/hudson/plugins/distfork/DistForkTask.java
          http://jenkins-ci.org/commit/distfork-plugin/34dc82ce67d15a63eed3a26993b728f956211b1e
          Log:
          JENKINS-23567 prevent logspam from distfork plugin.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: James Nord Path: src/main/java/hudson/plugins/distfork/DistForkTask.java http://jenkins-ci.org/commit/distfork-plugin/34dc82ce67d15a63eed3a26993b728f956211b1e Log: JENKINS-23567 prevent logspam from distfork plugin.

          Code changed in jenkins
          User: James Nord
          Path:
          src/main/java/hudson/plugins/distfork/DistForkTask.java
          http://jenkins-ci.org/commit/distfork-plugin/565fc5b2e655f30bbc632f79d9a72a1306e22e57
          Log:
          Merge pull request #3 from jtnord/jenkins-23567

          JENKINS-23567 prevent logspam from distfork plugin.

          Compare: https://github.com/jenkinsci/distfork-plugin/compare/aa63a739e677...565fc5b2e655

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: James Nord Path: src/main/java/hudson/plugins/distfork/DistForkTask.java http://jenkins-ci.org/commit/distfork-plugin/565fc5b2e655f30bbc632f79d9a72a1306e22e57 Log: Merge pull request #3 from jtnord/jenkins-23567 JENKINS-23567 prevent logspam from distfork plugin. Compare: https://github.com/jenkinsci/distfork-plugin/compare/aa63a739e677...565fc5b2e655

            Unassigned Unassigned
            terrapin Jan Hollevoet
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: