diff options
author | Harald Musum <musum@yahoo-inc.com> | 2016-07-06 09:16:05 +0200 |
---|---|---|
committer | Harald Musum <musum@yahoo-inc.com> | 2016-07-06 09:16:05 +0200 |
commit | 7ed7730f7050032e54779e26da588ab5d90c5bdc (patch) | |
tree | 359dcad0d7bd76e7aaaa10cf946a9d75e73cb9fd /node-admin | |
parent | dcc2c599d4b46c5f68ccc684aeabe1ae7d1bbd19 (diff) |
Update comment, node agents are removed in method
Diffstat (limited to 'node-admin')
-rw-r--r-- | node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeadmin/NodeAdminImpl.java | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeadmin/NodeAdminImpl.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeadmin/NodeAdminImpl.java index cde906dce45..e4f95f4c1c3 100644 --- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeadmin/NodeAdminImpl.java +++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeadmin/NodeAdminImpl.java @@ -160,11 +160,9 @@ public class NodeAdminImpl implements NodeAdmin { .map(key -> new Pair<>(Optional.ofNullable(tMap.get(key)), Optional.ofNullable(uMap.get(key)))); } - // TODO This method should rather take a lost of Hostname instead of Container. However, it triggers + // TODO This method should rather take a list of Hostname instead of Container. However, it triggers // a refactoring of the logic. Which is hard due to the style of programming. // The method streams the list of containers twice. - // It is not a full synchronization as it will only add new NodeAgent. Old ones are removed in - // garbageCollectDockerImages. We should refactor the code as some point. void synchronizeNodeSpecsToNodeAgents( final List<ContainerNodeSpec> containersToRun, final List<Container> existingContainers) { |