Purge LinkedLogRotator

It is O(n^2), and looks like it adds no real value. The normal rotator should delete all the builds.
jenkinsci · May 1, 2017 · c1b213b · c1b213b
1 parent ba659e0
commit c1b213b
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 94 deletions.
diff --git a/src/main/java/hudson/matrix/LinkedLogRotator.java b/src/main/java/hudson/matrix/LinkedLogRotator.java
diff --git a/src/main/java/hudson/matrix/MatrixConfiguration.java b/src/main/java/hudson/matrix/MatrixConfiguration.java
@@ -401,17 +401,6 @@ public Publisher getPublisher(Descriptor<Publisher> descriptor) {
         return getParent().getPublisher(descriptor);
     }
 
-    @Override
-    public BuildDiscarder getBuildDiscarder() {
-        // TODO: LinkedLogRotator doesn't work very well in the face of pluggable BuildDiscarder but I don't know what to do
-        BuildDiscarder bd = getParent().getBuildDiscarder();
-        if (bd instanceof LogRotator) {
-            LogRotator lr = (LogRotator) bd;
-            return new LinkedLogRotator(lr.getArtifactDaysToKeep(),lr.getArtifactNumToKeep());
-        }
-        return new LinkedLogRotator();
-    }
-
     @Override
     public SCM getScm() {
         return getParent().getScm();