Github user aledsage commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/810#discussion_r140602227
--- Diff:
core/src/main/java/org/apache/brooklyn/core/mgmt/ha/OsgiArchiveInstaller.java
---
@@ -60,6 +60,7 @@
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
+import com.google.common.base.Function;
--- End diff --
I only just noticed that a bunch of this code is in `core/mgmt/ha`. Why is
that? I'd have thought it would equally apply to non-ha (e.g. persisting your
state locally, and then restarting Brooklyn).
---