Obsolete Neon SR2 streams 50/102950/7
authorRobert Varga <robert.varga@pantheon.tech>
Fri, 28 Oct 2022 22:44:28 +0000 (00:44 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Mon, 6 Mar 2023 15:57:59 +0000 (16:57 +0100)
We nominally support reading Neon SR2 streams, but do not support writing
them out.

JIRA: YANGTOOLS-1492
Change-Id: Ia4655366357ce68f9dd18492f77c1f297658d26d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
15 files changed:
codec/yang-data-codec-binfmt/src/main/java/org/opendaylight/yangtools/yang/data/codec/binfmt/AbstractLithiumDataOutput.java [deleted file]
codec/yang-data-codec-binfmt/src/main/java/org/opendaylight/yangtools/yang/data/codec/binfmt/NeonSR2NormalizedNodeInputStreamReader.java
codec/yang-data-codec-binfmt/src/main/java/org/opendaylight/yangtools/yang/data/codec/binfmt/NeonSR2NormalizedNodeOutputStreamWriter.java [deleted file]
codec/yang-data-codec-binfmt/src/main/java/org/opendaylight/yangtools/yang/data/codec/binfmt/NormalizedNodeStreamVersion.java
codec/yang-data-codec-binfmt/src/test/java/org/opendaylight/yangtools/yang/data/codec/binfmt/AidSerializationTest.java
codec/yang-data-codec-binfmt/src/test/java/org/opendaylight/yangtools/yang/data/codec/binfmt/BitsSerializationTest.java
codec/yang-data-codec-binfmt/src/test/java/org/opendaylight/yangtools/yang/data/codec/binfmt/BooleanSerializationTest.java
codec/yang-data-codec-binfmt/src/test/java/org/opendaylight/yangtools/yang/data/codec/binfmt/BytesSerializationTest.java
codec/yang-data-codec-binfmt/src/test/java/org/opendaylight/yangtools/yang/data/codec/binfmt/IntSerializationTest.java
codec/yang-data-codec-binfmt/src/test/java/org/opendaylight/yangtools/yang/data/codec/binfmt/LithiumWriteObjectMappingTest.java [deleted file]
codec/yang-data-codec-binfmt/src/test/java/org/opendaylight/yangtools/yang/data/codec/binfmt/MapEntrySerializationTest.java
codec/yang-data-codec-binfmt/src/test/java/org/opendaylight/yangtools/yang/data/codec/binfmt/NipSerializationTest.java
codec/yang-data-codec-binfmt/src/test/java/org/opendaylight/yangtools/yang/data/codec/binfmt/NormalizedNodeStreamReaderWriterTest.java
codec/yang-data-codec-binfmt/src/test/java/org/opendaylight/yangtools/yang/data/codec/binfmt/StringSerializationTest.java
codec/yang-data-codec-binfmt/src/test/java/org/opendaylight/yangtools/yang/data/codec/binfmt/YiidSerializationTest.java

diff --git a/codec/yang-data-codec-binfmt/src/main/java/org/opendaylight/yangtools/yang/data/codec/binfmt/AbstractLithiumDataOutput.java b/codec/yang-data-codec-binfmt/src/main/java/org/opendaylight/yangtools/yang/data/codec/binfmt/AbstractLithiumDataOutput.java
deleted file mode 100644 (file)
index a0458ce..0000000
+++ /dev/null
@@ -1,400 +0,0 @@
-/*
- * Copyright (c) 2014 Cisco Systems, Inc. and others.  All rights reserved.
- *
- * This program and the accompanying materials are made available under the
- * terms of the Eclipse Public License v1.0 which accompanies this distribution,
- * and is available at http://www.eclipse.org/legal/epl-v10.html
- */
-package org.opendaylight.yangtools.yang.data.codec.binfmt;
-
-import static com.google.common.base.Preconditions.checkArgument;
-import static com.google.common.base.Preconditions.checkState;
-import static com.google.common.base.Verify.verifyNotNull;
-import static java.util.Objects.requireNonNull;
-
-import com.google.common.annotations.VisibleForTesting;
-import com.google.common.collect.ImmutableMap;
-import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
-import java.io.DataOutput;
-import java.io.IOException;
-import java.io.StringWriter;
-import java.math.BigDecimal;
-import java.math.BigInteger;
-import java.nio.charset.StandardCharsets;
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
-import java.util.Map.Entry;
-import java.util.Optional;
-import java.util.Set;
-import javax.xml.transform.TransformerException;
-import javax.xml.transform.TransformerFactory;
-import javax.xml.transform.dom.DOMSource;
-import javax.xml.transform.stream.StreamResult;
-import org.eclipse.jdt.annotation.NonNull;
-import org.opendaylight.yangtools.yang.common.Decimal64;
-import org.opendaylight.yangtools.yang.common.Empty;
-import org.opendaylight.yangtools.yang.common.QName;
-import org.opendaylight.yangtools.yang.common.QNameModule;
-import org.opendaylight.yangtools.yang.common.Revision;
-import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
-import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.AugmentationIdentifier;
-import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.NodeIdentifier;
-import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.NodeIdentifierWithPredicates;
-import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.NodeWithValue;
-import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.PathArgument;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
-/**
- * "original" type mapping. Baseline is Lithium but it really was introduced in Oxygen, where {@code type empty} was
- * remapped from null.
- *
- * <p>
- * {@code uint8}, {@code uint16}, {@code uint32} use java.lang types with widening, hence their value types overlap with
- * mapping of {@code int16}, {@code int32} and {@code int64}, making that difference indiscernible without YANG schema
- * knowledge.
- */
-abstract class AbstractLithiumDataOutput extends AbstractNormalizedNodeDataOutput {
-    private static final Logger LOG = LoggerFactory.getLogger(AbstractLithiumDataOutput.class);
-    private static final TransformerFactory TF = TransformerFactory.newInstance();
-    private static final ImmutableMap<Class<?>, Byte> KNOWN_TYPES = ImmutableMap.<Class<?>, Byte>builder()
-            .put(String.class, LithiumValue.STRING_TYPE)
-            .put(Byte.class, LithiumValue.BYTE_TYPE)
-            .put(Integer.class, LithiumValue.INT_TYPE)
-            .put(Long.class, LithiumValue.LONG_TYPE)
-            .put(Boolean.class, LithiumValue.BOOL_TYPE)
-            .put(QName.class, LithiumValue.QNAME_TYPE)
-            .put(Short.class, LithiumValue.SHORT_TYPE)
-            .put(BigInteger.class, LithiumValue.BIG_INTEGER_TYPE)
-            .put(BigDecimal.class, LithiumValue.BIG_DECIMAL_TYPE)
-            .put(Decimal64.class, LithiumValue.BIG_DECIMAL_TYPE)
-            .put(byte[].class, LithiumValue.BINARY_TYPE)
-            .put(Empty.class, LithiumValue.EMPTY_TYPE)
-            .build();
-
-    private final Map<String, Integer> stringCodeMap = new HashMap<>();
-
-    private QName lastLeafSetQName;
-    private boolean inSimple;
-
-    AbstractLithiumDataOutput(final DataOutput output) {
-        super(output);
-    }
-
-    @Override
-    public final void startLeafNode(final NodeIdentifier name) throws IOException {
-        LOG.trace("Starting a new leaf node");
-        startNode(name, LithiumNode.LEAF_NODE);
-        inSimple = true;
-    }
-
-    @Override
-    public final void startLeafSet(final NodeIdentifier name, final int childSizeHint) throws IOException {
-        LOG.trace("Starting a new leaf set");
-        commonStartLeafSet(name, LithiumNode.LEAF_SET);
-    }
-
-    @Override
-    public final void startOrderedLeafSet(final NodeIdentifier name, final int childSizeHint) throws IOException {
-        LOG.trace("Starting a new ordered leaf set");
-        commonStartLeafSet(name, LithiumNode.ORDERED_LEAF_SET);
-    }
-
-    private void commonStartLeafSet(final NodeIdentifier name, final byte nodeType) throws IOException {
-        startNode(name, nodeType);
-        lastLeafSetQName = name.getNodeType();
-    }
-
-    @Override
-    public final void startLeafSetEntryNode(final NodeWithValue<?> name) throws IOException {
-        LOG.trace("Starting a new leaf set entry node");
-
-        output.writeByte(LithiumNode.LEAF_SET_ENTRY_NODE);
-
-        // lastLeafSetQName is set if the parent LeafSetNode was previously written. Otherwise this is a
-        // stand alone LeafSetEntryNode so write out it's name here.
-        if (lastLeafSetQName == null) {
-            writeQNameInternal(name.getNodeType());
-        }
-        inSimple = true;
-    }
-
-    @Override
-    public final void startContainerNode(final NodeIdentifier name, final int childSizeHint) throws IOException {
-        LOG.trace("Starting a new container node");
-        startNode(name, LithiumNode.CONTAINER_NODE);
-    }
-
-    @Override
-    public final void startUnkeyedList(final NodeIdentifier name, final int childSizeHint) throws IOException {
-        LOG.trace("Starting a new unkeyed list");
-        startNode(name, LithiumNode.UNKEYED_LIST);
-    }
-
-    @Override
-    public final void startUnkeyedListItem(final NodeIdentifier name, final int childSizeHint) throws IOException {
-        LOG.trace("Starting a new unkeyed list item");
-        startNode(name, LithiumNode.UNKEYED_LIST_ITEM);
-    }
-
-    @Override
-    public final void startMapNode(final NodeIdentifier name, final int childSizeHint) throws IOException {
-        LOG.trace("Starting a new map node");
-        startNode(name, LithiumNode.MAP_NODE);
-    }
-
-    @Override
-    public final void startMapEntryNode(final NodeIdentifierWithPredicates identifier, final int childSizeHint)
-            throws IOException {
-        LOG.trace("Starting a new map entry node");
-        startNode(identifier, LithiumNode.MAP_ENTRY_NODE);
-        writeKeyValueMap(identifier.entrySet());
-    }
-
-    @Override
-    public final void startOrderedMapNode(final NodeIdentifier name, final int childSizeHint) throws IOException {
-        LOG.trace("Starting a new ordered map node");
-        startNode(name, LithiumNode.ORDERED_MAP_NODE);
-    }
-
-    @Override
-    public final void startChoiceNode(final NodeIdentifier name, final int childSizeHint) throws IOException {
-        LOG.trace("Starting a new choice node");
-        startNode(name, LithiumNode.CHOICE_NODE);
-    }
-
-    @Override
-    public final void startAugmentationNode(final AugmentationIdentifier identifier) throws IOException {
-        requireNonNull(identifier, "Node identifier should not be null");
-        LOG.trace("Starting a new augmentation node");
-
-        output.writeByte(LithiumNode.AUGMENTATION_NODE);
-        writeAugmentationIdentifier(identifier);
-    }
-
-    @Override
-    public final boolean startAnyxmlNode(final NodeIdentifier name, final Class<?> objectModel) throws IOException {
-        if (DOMSource.class.isAssignableFrom(objectModel)) {
-            LOG.trace("Starting anyxml node");
-            startNode(name, LithiumNode.ANY_XML_NODE);
-            inSimple = true;
-            return true;
-        }
-        return false;
-    }
-
-    @Override
-    public final void scalarValue(final Object value) throws IOException {
-        writeObject(value);
-    }
-
-    @Override
-    public final void domSourceValue(final DOMSource value) throws IOException {
-        final StringWriter writer = new StringWriter();
-        try {
-            TF.newTransformer().transform(value, new StreamResult(writer));
-        } catch (TransformerException e) {
-            throw new IOException("Error writing anyXml", e);
-        }
-        writeObject(writer.toString());
-    }
-
-    @Override
-    public final void endNode() throws IOException {
-        LOG.trace("Ending the node");
-        if (!inSimple) {
-            lastLeafSetQName = null;
-            output.writeByte(LithiumNode.END_NODE);
-        }
-        inSimple = false;
-    }
-
-    @Override
-    @SuppressFBWarnings(value = "BC_UNCONFIRMED_CAST",
-            justification = "The casts in the switch clauses are indirectly confirmed via the determination of 'type'.")
-    final void writePathArgumentInternal(final PathArgument pathArgument) throws IOException {
-        final byte type = LithiumPathArgument.getSerializablePathArgumentType(pathArgument);
-        output.writeByte(type);
-
-        switch (type) {
-            case LithiumPathArgument.NODE_IDENTIFIER:
-                NodeIdentifier nodeIdentifier = (NodeIdentifier) pathArgument;
-                writeQNameInternal(nodeIdentifier.getNodeType());
-                break;
-            case LithiumPathArgument.NODE_IDENTIFIER_WITH_PREDICATES:
-                NodeIdentifierWithPredicates nodeIdentifierWithPredicates =
-                    (NodeIdentifierWithPredicates) pathArgument;
-                writeQNameInternal(nodeIdentifierWithPredicates.getNodeType());
-                writeKeyValueMap(nodeIdentifierWithPredicates.entrySet());
-                break;
-            case LithiumPathArgument.NODE_IDENTIFIER_WITH_VALUE:
-                NodeWithValue<?> nodeWithValue = (NodeWithValue<?>) pathArgument;
-                writeQNameInternal(nodeWithValue.getNodeType());
-                writeObject(nodeWithValue.getValue());
-                break;
-            case LithiumPathArgument.AUGMENTATION_IDENTIFIER:
-                // No Qname in augmentation identifier
-                writeAugmentationIdentifier((AugmentationIdentifier) pathArgument);
-                break;
-            default:
-                throw new IllegalStateException("Unknown node identifier type is found : "
-                        + pathArgument.getClass().toString());
-        }
-    }
-
-    @Override
-    final void writeYangInstanceIdentifierInternal(final YangInstanceIdentifier identifier) throws IOException {
-        List<PathArgument> pathArguments = identifier.getPathArguments();
-        output.writeInt(pathArguments.size());
-
-        for (PathArgument pathArgument : pathArguments) {
-            writePathArgumentInternal(pathArgument);
-        }
-    }
-
-    final void defaultWriteAugmentationIdentifier(final @NonNull AugmentationIdentifier aid) throws IOException {
-        final Set<QName> qnames = aid.getPossibleChildNames();
-        // Write each child's qname separately, if list is empty send count as 0
-        if (!qnames.isEmpty()) {
-            output.writeInt(qnames.size());
-            for (QName qname : qnames) {
-                writeQNameInternal(qname);
-            }
-        } else {
-            LOG.debug("augmentation node does not have any child");
-            output.writeInt(0);
-        }
-    }
-
-    final void defaultWriteQName(final QName qname) throws IOException {
-        writeString(qname.getLocalName());
-        writeModule(qname.getModule());
-    }
-
-    final void defaultWriteModule(final QNameModule module) throws IOException {
-        writeString(module.getNamespace().toString());
-        final Optional<Revision> revision = module.getRevision();
-        if (revision.isPresent()) {
-            writeString(revision.get().toString());
-        } else {
-            writeByte(LithiumTokens.IS_NULL_VALUE);
-        }
-    }
-
-    abstract void writeModule(QNameModule module) throws IOException;
-
-    abstract void writeAugmentationIdentifier(@NonNull AugmentationIdentifier aid) throws IOException;
-
-    private void startNode(final PathArgument arg, final byte nodeType) throws IOException {
-        requireNonNull(arg, "Node identifier should not be null");
-        checkState(!inSimple, "Attempted to start a child in a simple node");
-
-        // First write the type of node
-        output.writeByte(nodeType);
-        // Write Start Tag
-        writeQNameInternal(arg.getNodeType());
-    }
-
-    private void writeObjSet(final Set<?> set) throws IOException {
-        output.writeInt(set.size());
-        for (Object o : set) {
-            checkArgument(o instanceof String, "Expected value type to be String but was %s (%s)", o.getClass(), o);
-            writeString((String) o);
-        }
-    }
-
-    private void writeObject(final Object value) throws IOException {
-        byte type = getSerializableType(value);
-        // Write object type first
-        output.writeByte(type);
-
-        switch (type) {
-            case LithiumValue.BOOL_TYPE:
-                output.writeBoolean((Boolean) value);
-                break;
-            case LithiumValue.QNAME_TYPE:
-                writeQNameInternal((QName) value);
-                break;
-            case LithiumValue.INT_TYPE:
-                output.writeInt((Integer) value);
-                break;
-            case LithiumValue.BYTE_TYPE:
-                output.writeByte((Byte) value);
-                break;
-            case LithiumValue.LONG_TYPE:
-                output.writeLong((Long) value);
-                break;
-            case LithiumValue.SHORT_TYPE:
-                output.writeShort((Short) value);
-                break;
-            case LithiumValue.BITS_TYPE:
-                writeObjSet((Set<?>) value);
-                break;
-            case LithiumValue.BINARY_TYPE:
-                byte[] bytes = (byte[]) value;
-                output.writeInt(bytes.length);
-                output.write(bytes);
-                break;
-            case LithiumValue.YANG_IDENTIFIER_TYPE:
-                writeYangInstanceIdentifierInternal((YangInstanceIdentifier) value);
-                break;
-            case LithiumValue.EMPTY_TYPE:
-                break;
-            case LithiumValue.STRING_BYTES_TYPE:
-                final byte[] valueBytes = value.toString().getBytes(StandardCharsets.UTF_8);
-                output.writeInt(valueBytes.length);
-                output.write(valueBytes);
-                break;
-            default:
-                output.writeUTF(value.toString());
-                break;
-        }
-    }
-
-    private void writeKeyValueMap(final Set<Entry<QName, Object>> entrySet) throws IOException {
-        if (!entrySet.isEmpty()) {
-            output.writeInt(entrySet.size());
-            for (Entry<QName, Object> entry : entrySet) {
-                writeQNameInternal(entry.getKey());
-                writeObject(entry.getValue());
-            }
-        } else {
-            output.writeInt(0);
-        }
-    }
-
-    private void writeString(final @NonNull String string) throws IOException {
-        final Integer value = stringCodeMap.get(verifyNotNull(string));
-        if (value == null) {
-            stringCodeMap.put(string, stringCodeMap.size());
-            writeByte(LithiumTokens.IS_STRING_VALUE);
-            writeUTF(string);
-        } else {
-            writeByte(LithiumTokens.IS_CODE_VALUE);
-            writeInt(value);
-        }
-    }
-
-    @VisibleForTesting
-    static final byte getSerializableType(final Object node) {
-        final Byte type = KNOWN_TYPES.get(requireNonNull(node).getClass());
-        if (type != null) {
-            if (type == LithiumValue.STRING_TYPE
-                    && ((String) node).length() >= LithiumValue.STRING_BYTES_LENGTH_THRESHOLD) {
-                return LithiumValue.STRING_BYTES_TYPE;
-            }
-            return type;
-        }
-
-        if (node instanceof Set) {
-            return LithiumValue.BITS_TYPE;
-        }
-
-        if (node instanceof YangInstanceIdentifier) {
-            return LithiumValue.YANG_IDENTIFIER_TYPE;
-        }
-
-        throw new IllegalArgumentException("Unknown value type " + node.getClass().getSimpleName());
-    }
-}
index f689fff59bab1a3c2ff27af25733fdcf25a31966..c46e7f36ba9cca30a8874646bf7eefe9ff81dffe 100644 (file)
@@ -33,6 +33,7 @@ final class NeonSR2NormalizedNodeInputStreamReader extends AbstractLithiumDataIn
     }
 
     @Override
+    @Deprecated(since = "11.0.0", forRemoval = true)
     public NormalizedNodeStreamVersion getVersion() {
         return NormalizedNodeStreamVersion.NEON_SR2;
     }
diff --git a/codec/yang-data-codec-binfmt/src/main/java/org/opendaylight/yangtools/yang/data/codec/binfmt/NeonSR2NormalizedNodeOutputStreamWriter.java b/codec/yang-data-codec-binfmt/src/main/java/org/opendaylight/yangtools/yang/data/codec/binfmt/NeonSR2NormalizedNodeOutputStreamWriter.java
deleted file mode 100644 (file)
index e9b3236..0000000
+++ /dev/null
@@ -1,90 +0,0 @@
-/*
- * Copyright (c) 2019 PANTHEON.tech, s.r.o. and others.  All rights reserved.
- *
- * This program and the accompanying materials are made available under the
- * terms of the Eclipse Public License v1.0 which accompanies this distribution,
- * and is available at http://www.eclipse.org/legal/epl-v10.html
- */
-package org.opendaylight.yangtools.yang.data.codec.binfmt;
-
-import java.io.DataOutput;
-import java.io.IOException;
-import java.util.HashMap;
-import java.util.Map;
-import org.opendaylight.yangtools.yang.common.QName;
-import org.opendaylight.yangtools.yang.common.QNameModule;
-import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.AugmentationIdentifier;
-
-/**
- * NormalizedNodeOutputStreamWriter will be used by distributed datastore to send normalized node in
- * a stream.
- * A stream writer wrapper around this class will write node objects to stream in recursive manner.
- * for example - If you have a ContainerNode which has a two LeafNode as children, then
- * you will first call
- * {@link #startContainerNode(org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.NodeIdentifier, int)},
- * then will call
- * {@link #leafNode(org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.NodeIdentifier, Object)} twice
- * and then, {@link #endNode()} to end container node.
- *
- * <p>Based on the each node, the node type is also written to the stream, that helps in reconstructing the object,
- * while reading.
- */
-final class NeonSR2NormalizedNodeOutputStreamWriter extends AbstractLithiumDataOutput {
-    private final Map<AugmentationIdentifier, Integer> aidCodeMap = new HashMap<>();
-    private final Map<QNameModule, Integer> moduleCodeMap = new HashMap<>();
-    private final Map<QName, Integer> qnameCodeMap = new HashMap<>();
-
-    NeonSR2NormalizedNodeOutputStreamWriter(final DataOutput output) {
-        super(output);
-    }
-
-    @Override
-    short streamVersion() {
-        return TokenTypes.NEON_SR2_VERSION;
-    }
-
-    @Override
-    void writeQNameInternal(final QName qname) throws IOException {
-        final Integer value = qnameCodeMap.get(qname);
-        if (value == null) {
-            // Fresh QName, remember it and emit as three strings
-            qnameCodeMap.put(qname, qnameCodeMap.size());
-            writeByte(NeonSR2Tokens.IS_QNAME_VALUE);
-            defaultWriteQName(qname);
-        } else {
-            // We have already seen this QName: write its code
-            writeByte(NeonSR2Tokens.IS_QNAME_CODE);
-            writeInt(value);
-        }
-    }
-
-    @Override
-    void writeAugmentationIdentifier(final AugmentationIdentifier aid) throws IOException {
-        final Integer value = aidCodeMap.get(aid);
-        if (value == null) {
-            // Fresh AugmentationIdentifier, remember it and emit as three strings
-            aidCodeMap.put(aid, aidCodeMap.size());
-            writeByte(NeonSR2Tokens.IS_AUGMENT_VALUE);
-            defaultWriteAugmentationIdentifier(aid);
-        } else {
-            // We have already seen this AugmentationIdentifier: write its code
-            writeByte(NeonSR2Tokens.IS_AUGMENT_CODE);
-            writeInt(value);
-        }
-    }
-
-    @Override
-    void writeModule(final QNameModule module) throws IOException {
-        final Integer value = moduleCodeMap.get(module);
-        if (value == null) {
-            // Fresh QNameModule, remember it and emit as three strings
-            moduleCodeMap.put(module, moduleCodeMap.size());
-            writeByte(NeonSR2Tokens.IS_MODULE_VALUE);
-            defaultWriteModule(module);
-        } else {
-            // We have already seen this QNameModule: write its code
-            writeByte(NeonSR2Tokens.IS_MODULE_CODE);
-            writeInt(value);
-        }
-    }
-}
index 15e67b3d19aa580f9f5b4ca802a1282aa3c813c1..a20335836954de20d8d6d4178ae1eb96f2a4fafd 100644 (file)
@@ -42,19 +42,22 @@ public enum NormalizedNodeStreamVersion {
     /**
      * Updated stream version, as shipped in OpenDaylight Neon SR2 release. Improves identifier encoding over
      * {@link #LITHIUM}, so that QName caching is more effective.
+     *
+     * @deprecated This version cannot be written in and always required adaption. This version should not be relied
+     *             upon, as it is subject to removal in a future version.
      */
+    @Deprecated(since = "11.0.0", forRemoval = true)
     NEON_SR2 {
         /**
          * {@inheritDoc}
-         *
-         * @deprecated This version is a historic one and should not be used in code. It does not support current
-         *             mapping of {@code Uint8} et al. and hence results in a stream which needs to be further adapted
-         *             to current definition of LeafNode.
+         * @implSpec
+         *     This method always throws {@link UnsupportedOperationException}.
+         * @deprecated This version is a historic one and writeout is not supported
          */
         @Override
         @Deprecated(since = "10.0.0", forRemoval = true)
         public NormalizedNodeDataOutput newDataOutput(final DataOutput output) {
-            return new NeonSR2NormalizedNodeOutputStreamWriter(output);
+            throw new UnsupportedOperationException();
         }
     },
     /**
index d11f287f9a5254eee4718fd84694f0144c30daf5..14b0bd583699105e7b2b04407d747db2e60c345d 100644 (file)
@@ -22,9 +22,8 @@ public class AidSerializationTest extends AbstractSerializationTest {
     @Parameters(name = "{0}")
     public static Iterable<Object[]> data() {
         return Arrays.asList(
-            new Object[] { NormalizedNodeStreamVersion.NEON_SR2,   9, 100, 421, 3145,   913_225,   913_231 },
-            new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1, 4,  94, 332, 2376,   716_618,   912_975 },
-            new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,  4,  94, 332, 2376,   716_618,   912_975 });
+            new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1, 4, 94, 332, 2376, 716_618, 912_975 },
+            new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,  4, 94, 332, 2376, 716_618, 912_975 });
     }
 
     @Parameter(1)
index f8957d8701ed7f04b5da32482e05a7cac3a5be34..e07323ce2d73749b01ad4931f6b3087200877694 100644 (file)
@@ -27,9 +27,8 @@ public class BitsSerializationTest extends AbstractSerializationTest {
     @Parameters(name = "{0}")
     public static Iterable<Object[]> data() {
         return Arrays.asList(
-            new Object[] { NormalizedNodeStreamVersion.NEON_SR2,   102, 106, 231, 1540, 456_766, 785_882 },
-            new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1,  96, 100, 226, 1536, 456_764, 654_045 },
-            new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,   96, 100, 226, 1536, 456_764, 654_045 });
+            new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1, 96, 100, 226, 1536, 456_764, 654_045 },
+            new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,  96, 100, 226, 1536, 456_764, 654_045 });
     }
 
     @Parameter(1)
index a35b9ed91749266fc1cd46ca956acba1a3a581bb..3a7b357001f091e5fa2191b125530b8b7aa0ba72 100644 (file)
@@ -19,7 +19,6 @@ public class BooleanSerializationTest extends AbstractSerializationTest {
     @Parameters(name = "{0}")
     public static Iterable<Object[]> data() {
         return Arrays.asList(
-            new Object[] { NormalizedNodeStreamVersion.NEON_SR2,   99 },
             new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1, 96 },
             new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,  96 });
     }
index 4c1bb07087557145d18e29d5874bd0fe6d2af2fa..8114ed0935404657e2daaddec005ff99e77774aa 100644 (file)
@@ -24,9 +24,8 @@ public class BytesSerializationTest extends AbstractSerializationTest {
     @Parameters(name = "{0}")
     public static Iterable<Object[]> data() {
         return Arrays.asList(
-            new Object[] { NormalizedNodeStreamVersion.NEON_SR2,   102, 103, 230, 486, 66022 },
-            new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1,  96,  97, 225, 482, 66020 },
-            new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,   96,  97, 225, 482, 66020 });
+            new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1, 96, 97, 225, 482, 66_020 },
+            new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,  96, 97, 225, 482, 66_020 });
     }
 
     @Parameter(1)
index 6faed8c34ea3eaf3067228fed6cedf651b602939..f2323ddb9bd78835f845ddb4c6ceadd7a2c3d646 100644 (file)
@@ -19,9 +19,8 @@ public class IntSerializationTest extends AbstractSerializationTest {
     @Parameters(name = "{0}")
     public static Iterable<Object[]> data() {
         return Arrays.asList(
-            new Object[] { NormalizedNodeStreamVersion.NEON_SR2,   99, 99, 100, 100, 102, 102, 102, 106, 106, 106 },
-            new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1, 96, 97, 96,   98,  96,  98, 100,  96, 100, 104 },
-            new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,  96, 97, 96,   98,  96,  98, 100,  96, 100, 104 });
+            new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1, 96, 97, 96, 98, 96, 98, 100, 96, 100, 104 },
+            new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,  96, 97, 96, 98, 96, 98, 100, 96, 100, 104 });
     }
 
     @Parameter(1)
diff --git a/codec/yang-data-codec-binfmt/src/test/java/org/opendaylight/yangtools/yang/data/codec/binfmt/LithiumWriteObjectMappingTest.java b/codec/yang-data-codec-binfmt/src/test/java/org/opendaylight/yangtools/yang/data/codec/binfmt/LithiumWriteObjectMappingTest.java
deleted file mode 100644 (file)
index c82d60b..0000000
+++ /dev/null
@@ -1,30 +0,0 @@
-/*
- * Copyright (c) 2015 Cisco Systems, Inc. and others.  All rights reserved.
- *
- * This program and the accompanying materials are made available under the
- * terms of the Eclipse Public License v1.0 which accompanies this distribution,
- * and is available at http://www.eclipse.org/legal/epl-v10.html
- */
-package org.opendaylight.yangtools.yang.data.codec.binfmt;
-
-import static org.junit.Assert.assertEquals;
-
-import org.junit.Test;
-
-public class LithiumWriteObjectMappingTest {
-    @Test
-    public void testStringType() {
-        assertEquals(LithiumValue.STRING_TYPE, AbstractLithiumDataOutput.getSerializableType("foobar"));
-        final String largeString = largeString(LithiumValue.STRING_BYTES_LENGTH_THRESHOLD);
-        assertEquals(LithiumValue.STRING_BYTES_TYPE, AbstractLithiumDataOutput.getSerializableType(largeString));
-    }
-
-    private static String largeString(final int minSize) {
-        final int pow = (int) (Math.log(minSize * 2) / Math.log(2));
-        StringBuilder sb = new StringBuilder("X");
-        for (int i = 0; i < pow; i++) {
-            sb.append(sb);
-        }
-        return sb.toString();
-    }
-}
index c3e03351a9b7db26e11a0a091a59229de704089e..c8e71f97843ae40c758c9e004aee7cfab05ec1bf 100644 (file)
@@ -27,9 +27,8 @@ public class MapEntrySerializationTest extends AbstractSerializationTest {
     @Parameters(name = "{0}")
     public static Iterable<Object[]> data() {
         return Arrays.asList(
-            new Object[] { NormalizedNodeStreamVersion.NEON_SR2,   102, 127, 152,  6_742 },
-            new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1,  96, 110, 125,  3_927 },
-            new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,   96, 110, 125,  3_927 });
+            new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1, 96, 110, 125, 3_927 },
+            new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,  96, 110, 125, 3_927 });
     }
 
     @Parameter(1)
index ef0ed856c5dea62a1913ee1c16954189e7a5dad0..e292f8503d10eb2e5727475802657ce50a3819b9 100644 (file)
@@ -27,9 +27,8 @@ public class NipSerializationTest extends AbstractSerializationTest {
     @Parameters(name = "{0}")
     public static Iterable<Object[]> data() {
         return Arrays.asList(
-            new Object[] { NormalizedNodeStreamVersion.NEON_SR2,   101, 116, 176, 4181, 1_180_245, 1_772_383 },
-            new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1,  95, 107, 156, 3409,   982_867, 1_443_164 },
-            new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,   95, 107, 156, 3409,   982_867, 1_443_164 });
+            new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1, 95, 107, 156, 3409, 982_867, 1_443_164 },
+            new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,  95, 107, 156, 3409, 982_867, 1_443_164 });
     }
 
     @Parameter(1)
index 07794ca5eb518325ee49108fb687183179f852a6..4270912e39c36a8a6248ab227b6cb4cd39e4eb45 100644 (file)
@@ -77,14 +77,12 @@ public class NormalizedNodeStreamReaderWriterTest {
     @Parameters(name = "{0} {1}")
     public static Iterable<Object[]> data() {
         return List.of(
-            new Object[] { NormalizedNodeStreamVersion.NEON_SR2,   Unsigned.BIG_INTEGER,
-                1_049_950, 5_577_993, 161, 1_163, 105, 235, 100 },
             new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1, Unsigned.BIG_INTEGER,
-                1_049_619, 2_289_103, 139,   826, 103, 229,  99 },
+                1_049_619, 2_289_103, 139, 826, 103, 229, 99 },
             new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1, Unsigned.UINT64,
-                1_049_618, 2_289_103, 139,   825, 103, 229,  99 },
+                1_049_618, 2_289_103, 139, 825, 103, 229, 99 },
             new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,  Unsigned.UINT64,
-                1_049_618, 2_289_103, 139,   825, 103, 229,  99 });
+                1_049_618, 2_289_103, 139, 825, 103, 229, 99 });
     }
 
     @Parameter(0)
index a4a20ab58ec5ef0728aca7447e56d2544bee4fa7..00a4b3bc786244ea9713950479143c1b589dc1b8 100644 (file)
@@ -7,8 +7,6 @@
  */
 package org.opendaylight.yangtools.yang.data.codec.binfmt;
 
-import static org.junit.Assume.assumeTrue;
-
 import java.util.Arrays;
 import org.junit.Test;
 import org.junit.runner.RunWith;
@@ -24,9 +22,8 @@ public class StringSerializationTest extends AbstractSerializationTest {
     @Parameters(name = "{0}")
     public static Iterable<Object[]> data() {
         return Arrays.asList(
-            new Object[] { NormalizedNodeStreamVersion.NEON_SR2,   100, 101, 32869, 33554534 },
-            new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1,  96,  99, 32865, 33554532 },
-            new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,   96,  99, 32865, 33554532 });
+            new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1, 96, 99, 32_865, 33_554_532 },
+            new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,  96, 99, 32_865, 33_554_532 });
     }
 
     @Parameter(1)
@@ -38,14 +35,8 @@ public class StringSerializationTest extends AbstractSerializationTest {
     @Parameter(4)
     public int hugeSize;
 
-    @Test
-    public void testEmptyString() {
-        assertEquals("", emptySize);
-    }
-
     @Test
     public void testEmptySame() {
-        assumeTrue(version.compareTo(NormalizedNodeStreamVersion.SODIUM_SR1) >= 0);
         assertSame("", emptySize);
     }
 
index a1086a60a9aa7507e69877baf38f96d3f9400400..2c12b7cc6f68055b4ae4e418f98904f68b8654e7 100644 (file)
@@ -26,14 +26,11 @@ public class YiidSerializationTest extends AbstractSerializationTest {
     @Parameters(name = "{0}")
     public static Iterable<Object[]> data() {
         return Arrays.asList(
-            new Object[] { NormalizedNodeStreamVersion.NEON_SR2,
-                102, 108, 288, 489, 294, 502, 1638, 3414,   394_854,   982_870, 1_377_611
-            },
             new Object[] { NormalizedNodeStreamVersion.SODIUM_SR1,
-                96,   98, 158, 359, 164, 372,  612, 2388,   131_684,   719_700,   916_815
+                96, 98, 158, 359, 164, 372, 612, 2388, 131_684, 719_700, 916_815
             },
             new Object[] { NormalizedNodeStreamVersion.MAGNESIUM,
-                96,   98, 158, 359, 164, 372,  612, 2388,   131_684 ,  719_700,   916_815
+                96, 98, 158, 359, 164, 372, 612, 2388, 131_684, 719_700, 916_815
             });
     }