summaryrefslogtreecommitdiffstats
path: root/container-core/src
diff options
context:
space:
mode:
authorgjoranv <gv@verizonmedia.com>2022-08-31 10:39:25 +0200
committergjoranv <gv@verizonmedia.com>2022-08-31 10:39:25 +0200
commit41b311dd2391e6d3d1566378ab120fa14424ed07 (patch)
tree34cf85d29b7917e08c8ff5fd3d3ba71219b87dca /container-core/src
parenta9459aeb4ad638ba05a90d3f8e8de24c3df47f9c (diff)
Add TODO for handling >2 generations of bundles.
Diffstat (limited to 'container-core/src')
-rw-r--r--container-core/src/main/java/com/yahoo/container/core/config/ApplicationBundleLoader.java7
1 files changed, 5 insertions, 2 deletions
diff --git a/container-core/src/main/java/com/yahoo/container/core/config/ApplicationBundleLoader.java b/container-core/src/main/java/com/yahoo/container/core/config/ApplicationBundleLoader.java
index 1e30b19a48d..19c3577d978 100644
--- a/container-core/src/main/java/com/yahoo/container/core/config/ApplicationBundleLoader.java
+++ b/container-core/src/main/java/com/yahoo/container/core/config/ApplicationBundleLoader.java
@@ -18,8 +18,11 @@ import java.util.logging.Logger;
/**
* Manages the set of installed and active/inactive bundles.
*
+ * TODO: This class and the CollisionHook currently only handles a "current" and "previous" generation.
+ * In order to correctly handle rapid reconfiguration and hence multiple generations, we need to
+ * consider the graph generation number for each bundle.
+ *
* @author gjoranv
- * @author Tony Vaagenes
*/
public class ApplicationBundleLoader {
@@ -117,7 +120,7 @@ public class ApplicationBundleLoader {
}
private Map<FileReference, Bundle> installWithFileDistribution(Set<FileReference> bundlesToInstall,
- FileAcquirerBundleInstaller bundleInstaller) {
+ FileAcquirerBundleInstaller bundleInstaller) {
var newBundles = new LinkedHashMap<FileReference, Bundle>();
for (FileReference reference : bundlesToInstall) {