X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=openflow-protocol-impl%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fopenflowjava%2Fprotocol%2Fimpl%2Fserialization%2FSerializerRegistryImpl.java;h=26cdda1a0b3e80f4894230326c9b7d3e4d014110;hb=29a2a074c78708f6d18583779ece96bb6573f0c6;hp=e8a02bd78e4811743f2f74e7606d2277c5e24e4e;hpb=4892efadc4ef9a61831828bddfe0e939b4334779;p=openflowjava.git
diff --git a/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/SerializerRegistryImpl.java b/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/SerializerRegistryImpl.java
index e8a02bd7..26cdda1a 100644
--- a/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/SerializerRegistryImpl.java
+++ b/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/SerializerRegistryImpl.java
@@ -11,23 +11,28 @@ package org.opendaylight.openflowjava.protocol.impl.serialization;
import java.util.HashMap;
import java.util.Map;
-import org.opendaylight.openflowjava.protocol.api.extensibility.MessageTypeKey;
import org.opendaylight.openflowjava.protocol.api.extensibility.OFGeneralSerializer;
import org.opendaylight.openflowjava.protocol.api.extensibility.SerializerRegistry;
import org.opendaylight.openflowjava.protocol.api.extensibility.SerializerRegistryInjector;
+import org.opendaylight.openflowjava.protocol.api.keys.MessageTypeKey;
import org.opendaylight.openflowjava.protocol.api.util.EncodeConstants;
import org.opendaylight.openflowjava.protocol.impl.util.OF10MatchSerializer;
import org.opendaylight.openflowjava.protocol.impl.util.OF13MatchSerializer;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.oxm.rev130731.match.grouping.Match;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.oxm.rev130731.match.v10.grouping.MatchV10;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.oxm.rev150225.match.grouping.Match;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.oxm.rev150225.match.v10.grouping.MatchV10;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
/**
- * Stores and handles serializers
+ * Stores and handles serializers
+ * K - {@link MessageTypeKey} type
+ * S - returned serializer type
* @author michal.polkorab
* @author timotej.kubas
*/
public class SerializerRegistryImpl implements SerializerRegistry {
+ private static final Logger LOGGER = LoggerFactory.getLogger(SerializerRegistryImpl.class);
private static final short OF10 = EncodeConstants.OF10_VERSION_ID;
private static final short OF13 = EncodeConstants.OF13_VERSION_ID;
private Map, OFGeneralSerializer> registry;
@@ -57,31 +62,36 @@ public class SerializerRegistryImpl implements SerializerRegistry {
*/
@Override
@SuppressWarnings("unchecked")
- public SERIALIZER_TYPE getSerializer(
- MessageTypeKey msgTypeKey) {
+ public S getSerializer(
+ MessageTypeKey msgTypeKey) {
OFGeneralSerializer serializer = registry.get(msgTypeKey);
if (serializer == null) {
- throw new IllegalArgumentException("Serializer for key: {}" + msgTypeKey.toString()
+ throw new IllegalStateException("Serializer for key: " + msgTypeKey
+ " was not found - please verify that you are using correct message"
+ " combination (e.g. OF v1.0 message to OF v1.0 device)");
}
- return (SERIALIZER_TYPE) serializer;
+ return (S) serializer;
}
@Override
- public void registerSerializer(
- MessageTypeKey msgTypeKey, OFGeneralSerializer serializer) {
+ public void registerSerializer(
+ MessageTypeKey msgTypeKey, OFGeneralSerializer serializer) {
if ((msgTypeKey == null) || (serializer == null)) {
throw new IllegalArgumentException("MessageTypeKey or Serializer is null");
}
+ OFGeneralSerializer serInRegistry = registry.put(msgTypeKey, serializer);
+ if (serInRegistry != null) {
+ LOGGER.debug("Serializer for key " + msgTypeKey + " overwritten. Old serializer: "
+ + serInRegistry.getClass().getName() + ", new serializer: "
+ + serializer.getClass().getName() );
+ }
if (serializer instanceof SerializerRegistryInjector) {
((SerializerRegistryInjector) serializer).injectSerializerRegistry(this);
}
- registry.put(msgTypeKey, serializer);
}
@Override
- public boolean unregisterSerializer(MessageTypeKey msgTypeKey) {
+ public boolean unregisterSerializer(MessageTypeKey msgTypeKey) {
if (msgTypeKey == null) {
throw new IllegalArgumentException("MessageTypeKey is null");
}