summaryrefslogtreecommitdiffstats
path: root/bundle-plugin/src/main/java/com/yahoo/container/plugin/classanalysis/Packages.java
diff options
context:
space:
mode:
Diffstat (limited to 'bundle-plugin/src/main/java/com/yahoo/container/plugin/classanalysis/Packages.java')
-rw-r--r--bundle-plugin/src/main/java/com/yahoo/container/plugin/classanalysis/Packages.java17
1 files changed, 7 insertions, 10 deletions
diff --git a/bundle-plugin/src/main/java/com/yahoo/container/plugin/classanalysis/Packages.java b/bundle-plugin/src/main/java/com/yahoo/container/plugin/classanalysis/Packages.java
index 48a128c2f0d..a9f95f2b2f2 100644
--- a/bundle-plugin/src/main/java/com/yahoo/container/plugin/classanalysis/Packages.java
+++ b/bundle-plugin/src/main/java/com/yahoo/container/plugin/classanalysis/Packages.java
@@ -7,7 +7,6 @@ import java.util.HashSet;
import java.util.List;
import java.util.Map;
import java.util.Set;
-import java.util.stream.Collectors;
/**
* Utility methods related to packages.
@@ -37,19 +36,17 @@ public class Packages {
}
/**
- * Returns the imported Vespa packages that don't exist in the given set of allowed packages.
+ * Returns the imported packages that exist in the given set of disallowed packages.
*/
- public static List<String> disallowedVespaImports(Map<String, ImportPackages.Import> imports, List<String> allowed) {
+ public static List<String> disallowedImports(Map<String, ImportPackages.Import> imports, List<String> disallowed) {
if (imports == null || imports.isEmpty()) return List.of();
- var publicApi = allowed == null ? Set.of() : new HashSet<>(allowed);
+ var importedSet = new HashSet<>(imports.keySet());
+ var disallowedSet = disallowed == null ? Set.of() : new HashSet<>(disallowed);
- Set<String> yahooImports = imports.keySet().stream()
- .filter(pkg -> pkg.startsWith("com.yahoo") || pkg.startsWith("ai.vespa."))
- .collect(Collectors.toSet());
-
- List<String> disallowedImports = yahooImports.stream().collect(Collectors.groupingBy(publicApi::contains)).get(false);
- return disallowedImports == null ? List.of() : disallowedImports;
+ return importedSet.stream()
+ .filter(disallowedSet::contains)
+ .toList();
}
public static PackageMetaData analyzePackages(Set<ClassFileMetaData> allClasses) {