diff options
author | gjoranv <gv@verizonmedia.com> | 2022-08-31 10:39:25 +0200 |
---|---|---|
committer | gjoranv <gv@verizonmedia.com> | 2022-08-31 10:39:25 +0200 |
commit | 41b311dd2391e6d3d1566378ab120fa14424ed07 (patch) | |
tree | 34cf85d29b7917e08c8ff5fd3d3ba71219b87dca /container-core | |
parent | a9459aeb4ad638ba05a90d3f8e8de24c3df47f9c (diff) |
Add TODO for handling >2 generations of bundles.
Diffstat (limited to 'container-core')
-rw-r--r-- | container-core/src/main/java/com/yahoo/container/core/config/ApplicationBundleLoader.java | 7 |
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) { |