summaryrefslogtreecommitdiffstats
path: root/container-core
diff options
context:
space:
mode:
authorgjoranv <gv@verizonmedia.com>2019-10-29 11:03:57 +0100
committergjoranv <gv@verizonmedia.com>2019-10-29 11:03:57 +0100
commitfb652e337235245740602a6c586a668e98f68993 (patch)
tree8e0134aa681d8b6b0cec54e06cd1dc1dad6f9b6d /container-core
parent0846f9d3277ab1965f9718045a89ec2865128852 (diff)
Update comments and remove an import.
Diffstat (limited to 'container-core')
-rw-r--r--container-core/src/main/java/com/yahoo/container/core/config/BundleLoader.java4
1 files changed, 1 insertions, 3 deletions
diff --git a/container-core/src/main/java/com/yahoo/container/core/config/BundleLoader.java b/container-core/src/main/java/com/yahoo/container/core/config/BundleLoader.java
index c796b773232..18886d2e5a9 100644
--- a/container-core/src/main/java/com/yahoo/container/core/config/BundleLoader.java
+++ b/container-core/src/main/java/com/yahoo/container/core/config/BundleLoader.java
@@ -10,7 +10,6 @@ import org.osgi.framework.Bundle;
import org.osgi.framework.wiring.BundleRevision;
import java.io.File;
-import java.util.Arrays;
import java.util.HashSet;
import java.util.LinkedHashMap;
import java.util.List;
@@ -52,12 +51,11 @@ public class BundleLoader {
return osgi.install(file.getAbsolutePath());
}
- /** Returns the number of bundles installed by this call. */
private void install(List<FileReference> references) {
Set<FileReference> bundlesToInstall = new HashSet<>(references);
// This is just an optimization, as installing a bundle with the same location id returns the already installed bundle.
- // It has no effect that pendingUninstall fileRefs are removed from the map, as they are NOT in the new set of bundles..
+ // It's ok that fileRefs pending uninstall are removed from the map, because they are never in the new set of bundles..
bundlesToInstall.removeAll(reference2Bundles.keySet());
PredicateSplit<FileReference> bundlesToInstall_isDisk = partition(bundlesToInstall, BundleLoader::isDiskBundle);