From f5aa814e4054827db30643863e18a29e6606c749 Mon Sep 17 00:00:00 2001 From: Kanzhe Jiang <kanzhe.jiang@bigswitch.com> Date: Tue, 5 Jun 2012 15:38:37 -0700 Subject: [PATCH] More documentation in the new topologyService method, getLastLinkUpdates() --- .../floodlightcontroller/topology/ITopologyService.java | 7 ++++--- .../net/floodlightcontroller/topology/TopologyManager.java | 2 +- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/src/main/java/net/floodlightcontroller/topology/ITopologyService.java b/src/main/java/net/floodlightcontroller/topology/ITopologyService.java index 96a729c38..6b38f60be 100644 --- a/src/main/java/net/floodlightcontroller/topology/ITopologyService.java +++ b/src/main/java/net/floodlightcontroller/topology/ITopologyService.java @@ -185,9 +185,10 @@ public interface ITopologyService extends IFloodlightService { public Set<NodePortTuple> getBlockedPorts(); /** - * Returns a set of link updates, which had been applied - * in computing the new topology. + * ITopologyListener provides topologyChanged notification, + * but not *what* the changes were. + * This method returns the delta in the linkUpdates between the current and the previous topology instance. * @return */ - public Set<LDUpdate> getAppliedLinkUpdates(); + public Set<LDUpdate> getLastLinkUpdates(); } diff --git a/src/main/java/net/floodlightcontroller/topology/TopologyManager.java b/src/main/java/net/floodlightcontroller/topology/TopologyManager.java index 54aafaa25..e823e55dc 100644 --- a/src/main/java/net/floodlightcontroller/topology/TopologyManager.java +++ b/src/main/java/net/floodlightcontroller/topology/TopologyManager.java @@ -429,7 +429,7 @@ public class TopologyManager implements } @Override - public Set<LDUpdate> getAppliedLinkUpdates() { + public Set<LDUpdate> getLastLinkUpdates() { return appliedUpdates; } //////////////////////////////////////////////////////////////////////// -- GitLab