Removing invalid casts from hosttracker. 42/442/1
authorMadhu Venugopal <vmadhu@cisco.com>
Fri, 7 Jun 2013 04:21:45 +0000 (21:21 -0700)
committerMadhu Venugopal <vmadhu@cisco.com>
Fri, 7 Jun 2013 04:23:31 +0000 (21:23 -0700)
Signed-off-by: Madhu Venugopal <vmadhu@cisco.com>
Change-Id: I4a2c7e73a025112ee77a2b0f7058a4d6d593c76a
Signed-off-by: Madhu Venugopal <vmadhu@cisco.com>
opendaylight/hosttracker/api/src/main/java/org/opendaylight/controller/hosttracker/hostAware/HostNodeConnector.java
opendaylight/hosttracker/api/src/test/java/org/opendaylight/controller/hosttracker/hostAware/HostNodeConnectorTest.java
opendaylight/hosttracker/implementation/src/main/java/org/opendaylight/controller/hosttracker/internal/HostTracker.java

index e7b5745a2ec63c3fc3544d3b538e1d5188154303..243da05869a5edd4ec7d41eedccf61b2b5e1079f 100644 (file)
@@ -82,20 +82,6 @@ public class HostNodeConnector extends Host {
         return this.nodeConnector.getNode();
     }
 
-    /**
-     * @return the NodeId
-     */
-    public Long getnodeconnectornodeId() {
-        return (Long) this.nodeConnector.getNode().getID();
-    }
-
-    /**
-     * @return the port
-     */
-    public Short getnodeconnectorportId() {
-        return (Short) this.nodeConnector.getID();
-    }
-
     /**
      * @return the DataLayerAddress
      */
index ee38fc72e819a1d2d229d7f54525b0ca079ed64c..c29e5b16453407e8999bdbaec21cd9eabc9185a5 100644 (file)
@@ -57,8 +57,7 @@ public class HostNodeConnectorTest extends TestCase {
                        Assert.assertTrue(hostnodeconnector_2.isV6Host());\r
                        Assert.assertTrue(hostnodeconnector_2.getnodeConnector().equals(nc1));\r
                        Assert.assertTrue(hostnodeconnector_2.getnodeconnectorNode().equals(node));\r
-                       Assert.assertTrue(node.getID().equals(hostnodeconnector_2.getnodeconnectornodeId()));\r
-                       Assert.assertTrue(hostnodeconnector_2.getnodeconnectorportId().equals((short)2));\r
+                        Assert.assertTrue(node.equals(hostnodeconnector_2.getnodeconnectorNode()));\r
                } catch (ConstructionException e) {\r
                        Assert.assertTrue(false);\r
                }\r
index 9096797553364e90b8db6116c3ef0e210408b45f..f8534e98101702b6ce1039714eb9a647392cf85f 100644 (file)
@@ -1255,18 +1255,12 @@ public class HostTracker implements IfIptoHost, IfHostListener,
     }
 
     private void handleNodeConnectorStatusDown(NodeConnector nodeConnector) {
-        long sid = (Long) nodeConnector.getNode().getID();
-        short port = (Short) nodeConnector.getID();
-
         logger.debug("handleNodeConnectorStatusDown {}", nodeConnector);
 
         for (Entry<InetAddress, HostNodeConnector> entry : hostsDB.entrySet()) {
             HostNodeConnector host = entry.getValue();
-            if ((host.getnodeconnectornodeId() == sid)
-                    && (host.getnodeconnectorportId() == port)) {
-                logger.debug(
-                        "Switch: {}, Port: {} is down, remove from Hosts_DB",
-                        sid, port);
+            if (host.getnodeConnector().equals(nodeConnector)) {
+                logger.debug(" NodeConnector: {} is down, remove from Hosts_DB", nodeConnector);
                 removeKnownHost(entry.getKey());
                 notifyHostLearnedOrRemoved(host, false);
             }