commit c25861af883aebb695066d1da1c33e60b8776115
author Yuta HIGUCHI <y-higuchi@onlab.us> Tue Jun 17 23:17:09 2014 -0700
committer Gerrit Code Review <gerrit2@gerrit.onos.onlab.us> Fri Jun 20 10:33:06 2014 -0700
tree 890d07dbe4c52321d25bb287ae5da927aa0cc702
parent ee81899f39a4721c5344710b12a14caf1ea86fea

Cleaning KryoFactory registration

- Add LinkedList potentially used in DeviceEvent
- Remove unused class from registration.
  - DataPathEndpoints
     not referenced from anywhere in tree,
     class itself can probably be removed

Change-Id: Ia3755ad7b09c9f3495dccf0ed873712b38ccc2e9

src/main/java/net/onrc/onos/core/util/serializers/KryoFactory.java

diff --git a/src/main/java/net/onrc/onos/core/util/serializers/KryoFactory.java b/src/main/java/net/onrc/onos/core/util/serializers/KryoFactory.java
index a01cfc4..3b06ec3 100644
--- a/src/main/java/net/onrc/onos/core/util/serializers/KryoFactory.java
+++ b/src/main/java/net/onrc/onos/core/util/serializers/KryoFactory.java
@@ -6,6 +6,7 @@
 import java.util.Date;
 import java.util.HashMap;
 import java.util.HashSet;
+import java.util.LinkedList;
 
 import net.floodlightcontroller.util.MACAddress;
 import net.onrc.onos.apps.proxyarp.ArpCacheNotification;
@@ -30,7 +31,6 @@
 import net.onrc.onos.core.topology.TopologyEvent;
 import net.onrc.onos.core.util.CallerId;
 import net.onrc.onos.core.util.DataPath;
-import net.onrc.onos.core.util.DataPathEndpoints;
 import net.onrc.onos.core.util.Dpid;
 import net.onrc.onos.core.util.FlowEntry;
 import net.onrc.onos.core.util.FlowEntryAction;
@@ -51,7 +51,6 @@
 import net.onrc.onos.core.util.IPv6Net;
 import net.onrc.onos.core.util.Port;
 import net.onrc.onos.core.util.Switch;
-// import net.onrc.onos.core.util.SwitchPort;
 
 import com.esotericsoftware.kryo.Kryo;
 
@@ -152,7 +151,6 @@
         // FlowPath and related classes
         kryo.register(CallerId.class);
         kryo.register(DataPath.class);
-        kryo.register(DataPathEndpoints.class);
         kryo.register(Dpid.class);
         kryo.register(FlowEntryAction.class);
         kryo.register(FlowEntryAction.ActionEnqueue.class);
@@ -190,7 +188,7 @@
 
         // New data model-related classes
         kryo.register(DeviceEvent.class);
-        kryo.register(InetAddress.class);
+        kryo.register(LinkedList.class);
         kryo.register(LinkEvent.class);
         kryo.register(PortEvent.class);
         kryo.register(PortEvent.SwitchPort.class);
@@ -214,7 +212,6 @@
 
         // Device-related classes
         kryo.register(HashSet.class);
-        kryo.register(Inet4Address.class);
         kryo.register(OnosDevice.class);
         kryo.register(Date.class);
 
@@ -224,6 +221,10 @@
         kryo.register(SinglePacketOutNotification.class);
         kryo.register(ArpReplyNotification.class);
         kryo.register(ArpCacheNotification.class);
+        // TODO check if InetAddress related is still used
+        // TODO check if InetAddress can be correctly serialized
+        kryo.register(InetAddress.class);
+        kryo.register(Inet4Address.class);
 
         return kryo;
     }