diff options
author | gjoranv <gv@verizonmedia.com> | 2022-09-08 14:19:55 +0200 |
---|---|---|
committer | gjoranv <gv@verizonmedia.com> | 2022-09-08 14:19:55 +0200 |
commit | 62ed862f1f66350173a8e0606d2e5f25fab83b93 (patch) | |
tree | 45f8efe03d44626923a0ae7a7666e08c7e665ab5 | |
parent | c4ffdbdd935b6f5e7876a5aed5d2a9f571c5a85c (diff) |
Update stale comments after refactoring the code.
-rw-r--r-- | container-core/src/main/java/com/yahoo/container/core/config/ApplicationBundleLoader.java | 9 |
1 files changed, 5 insertions, 4 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 88e66c45ca3..cf3db91239a 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 @@ -66,6 +66,9 @@ public class ApplicationBundleLoader { readyForNewBundles = false; } + /** + * Must be called after useBundles() to report success or failure for the latest bundle generation. + */ public synchronized Set<Bundle> completeGeneration(GenerationStatus status) { Set<Bundle> ret = Set.of(); if (readyForNewBundles) return ret; @@ -79,10 +82,8 @@ public class ApplicationBundleLoader { } /** - * Commit to the current set of bundles. Must be called after the component graph creation proved successful, - * to prevent uninstalling bundles unintentionally upon a future call to {@link #revertToPreviousGeneration()}. - * Returns the set of bundles that is no longer used by the application, and should therefore be scheduled - * for uninstall. + * Commit to the latest set of bundles given to useBundles(). Returns the set of bundles that is no longer + * used by the application, and should therefore be scheduled for uninstall. */ private Set<Bundle> commitBundles() { var bundlesToUninstall = new LinkedHashSet<>(obsoleteBundles.values()); |