Merge "bug 3126 - shift to new notification service"

[openflowplugin.git] / openflowplugin-impl / src / main / java / org / opendaylight / openflowplugin / impl / registry / flow / FlowHashFactory.java

diff --git a/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/registry/flow/FlowHashFactory.java b/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/registry/flow/FlowHashFactory.java
index 47f73360027a9f8581fc6bd5ab817f07d0d3ab0e..9747d014409ea774bd0fd9f3616431c627bee453 100644 (file)
--- a/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/registry/flow/FlowHashFactory.java
+++ b/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/registry/flow/FlowHashFactory.java
@@ -8,6 +8,10 @@
 
 package org.opendaylight.openflowplugin.impl.registry.flow;
 
+import com.google.common.primitives.Longs;
+import java.math.BigInteger;
+import java.util.Objects;
+import org.opendaylight.openflowplugin.api.openflow.device.DeviceContext;
 import org.opendaylight.openflowplugin.api.openflow.registry.flow.FlowHash;
 import org.opendaylight.openflowplugin.impl.util.HashUtil;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.Flow;
@@ -21,31 +25,37 @@ public class FlowHashFactory {
     public FlowHashFactory() {
     }
 
-    public static FlowHash create(Flow flow) {
-        int hash = calculateHash(flow);
-        return new FlowHashDto(hash);
+    public static FlowHash create(Flow flow, DeviceContext deviceContext) {
+        long hash = calculateHash(flow, deviceContext);
+        return new FlowHashDto(hash, flow);
     }
 
-    private static int calculateHash(Flow flow) {
-        int hash = HashUtil.calculateMatchHash(flow.getMatch());
-        hash += flow.getPriority();
-        hash += flow.getTableId();
-        hash += flow.getCookie().hashCode();
-        return hash;
+    private static long calculateHash(Flow flow, DeviceContext deviceContext) {
+        return HashUtil.calculateMatchHash(flow.getMatch(), deviceContext);
     }
 
 
     private static final class FlowHashDto implements FlowHash {
 
-        private final int hashCode;
+        private final long flowHash;
+        private final int intHashCode;
 
-        public FlowHashDto(final int hashCode) {
-            this.hashCode = hashCode;
+        private final short tableId;
+        private final int priority;
+        private final BigInteger cookie;
+
+        public FlowHashDto(final long flowHash, final Flow flow) {
+            this.flowHash = flowHash;
+            this.intHashCode = Longs.hashCode(flowHash);
+            tableId = flow.getTableId();
+            priority = flow.getPriority();
+            cookie = flow.getCookie().getValue();
         }
 
+
         @Override
         public int hashCode() {
-            return hashCode;
+            return intHashCode;
         }
 
         @Override
@@ -56,12 +66,34 @@ public class FlowHashFactory {
             if (!(obj instanceof FlowHash)) {
                 return false;
             }
-            if (this.hashCode == obj.hashCode()) {
+            FlowHash that = (FlowHash) obj;
+            if (this.flowHash == that.getFlowHash()
+                    && this.tableId == that.getTableId()
+                    && this.priority == that.getPriority()
+                    && Objects.equals(this.cookie, that.getCookie())) {
                 return true;
             }
             return false;
         }
 
+        @Override
+        public long getFlowHash() {
+            return flowHash;
+        }
+
+        @Override
+        public short getTableId() {
+            return tableId;
+        }
+
+        @Override
+        public int getPriority() {
+            return priority;
+        }
 
+        @Override
+        public BigInteger getCookie() {
+            return cookie;
+        }
     }
 }