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

Use of beta Guava method since deleted: Iterators.skip

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • core

      Jenkins core currently bundles Guava 11. I found that if a plugin which is using a newer release of Guava, by masking Guava classes from core or using pluginFirstClassLoader, runs certain kinds of functional tests, it can get linkage errors:

      java.lang.NoSuchMethodError: com.google.common.collect.Iterators.skip(Ljava/util/Iterator;I)I
      	at hudson.util.RunList.subList(RunList.java:153)
      	at hudson.tasks.LogRotator.perform(LogRotator.java:153)
      	at hudson.model.Job.logRotate(Job.java:468)
      	at ...
      

      Iterators.skip is @Beta in Guava 11 and was deleted in Guava 14. Anyway it is a pretty trivial utility method, so it can just go into hudson.util.Iterators or be inlined.

          [JENKINS-51779] Use of beta Guava method since deleted: Iterators.skip

          Jesse Glick created issue -
          Jesse Glick made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Jesse Glick made changes -
          Link New: This issue relates to JENKINS-41827 [ JENKINS-41827 ]
          Jesse Glick made changes -
          Link New: This issue blocks JENKINS-50520 [ JENKINS-50520 ]
          Jesse Glick made changes -
          Labels New: lts-candidate
          Jesse Glick made changes -
          Remote Link New: This issue links to "PR 3481 (Web Link)" [ 20820 ]
          Jesse Glick made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Jesse Glick made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Review [ 10005 ] New: Resolved [ 5 ]
          Jesse Glick made changes -
          Link New: This issue relates to JENKINS-51889 [ JENKINS-51889 ]
          Oliver Gondža made changes -
          Labels Original: lts-candidate New: 2.121.2-fixed
          Jesse Glick made changes -
          Remote Link New: This issue links to "artifact-manager-s3 PR 77 (Web Link)" [ 21417 ]

            jglick Jesse Glick
            jglick Jesse Glick
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: