Merge pull request #292 from GoodOwl/faster-path-resolver

Improve performance of relative path mapping
jenkinsci · Dec 11, 2024 · 0949826 · 0949826
2 parents 652f519 + ba75ea4
commit 0949826
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/plugin/src/main/java/io/jenkins/plugins/coverage/metrics/steps/CoverageRecorder.java b/plugin/src/main/java/io/jenkins/plugins/coverage/metrics/steps/CoverageRecorder.java
@@ -445,9 +445,13 @@ private void resolveAbsolutePaths(final Node rootNode, final FilePath workspace,
         if (!pathMapping.isEmpty()) {
             log.logInfo("Making paths of " + pathMapping.size() + " source code files relative to workspace root...");
             var builder = new TreeStringBuilder();
-            rootNode.getAllFileNodes().stream()
-                    .filter(file -> pathMapping.containsKey(file.getRelativePath()))
-                    .forEach(file -> file.setRelativePath(builder.intern(pathMapping.get(file.getRelativePath()))));
+            rootNode.getAllFileNodes().forEach(file -> {
+                String relativePath = file.getRelativePath();
+                if (pathMapping.containsKey(relativePath)) {
+                    file.setRelativePath(builder.intern(pathMapping.get(relativePath)));
+                }
+            });
+
             builder.dedup();
         }
     }