Use a simple list for string code table 89/80889/1
authorRobert Varga <robert.varga@pantheon.tech>
Fri, 15 Mar 2019 11:33:05 +0000 (12:33 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Fri, 15 Mar 2019 11:33:05 +0000 (12:33 +0100)
Since the codes are assigned in a linear fashion, we can use
a simple ArrayList to track assignes codes, preventing the need
for boxing integer during population and lookups. Lookups also
end up requiring an array offset lookup rather than needing
to go through multiple indirections.

While this can in theory lead to a large array being allocated,
the set of strings we keep in the table is limited by the
SchemaContext and thus will be typically capped ad couple of
thousand.

The cost of growing the lookup should not be significantly
higher, even if ArrayList grows slower:
- 10, 16, 25, 38,  58,  88
versus HashMap (with .75 load factor)
- 12, 24, 48, 96, 192, 384

each resize operation is significantly faster.

Change-Id: I00c7a2e0985dbf901b6857a16424bf157e5af39d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
opendaylight/md-sal/sal-clustering-commons/src/main/java/org/opendaylight/controller/cluster/datastore/node/utils/stream/NormalizedNodeInputStreamReader.java

index 0ea2fe7..b042963 100755 (executable)
@@ -18,9 +18,9 @@ import java.io.StringReader;
 import java.math.BigDecimal;
 import java.math.BigInteger;
 import java.nio.charset.StandardCharsets;
-import java.util.HashMap;
+import java.util.ArrayList;
 import java.util.HashSet;
-import java.util.Map;
+import java.util.List;
 import java.util.Set;
 import javax.xml.parsers.DocumentBuilderFactory;
 import javax.xml.parsers.ParserConfigurationException;
@@ -61,7 +61,7 @@ public class NormalizedNodeInputStreamReader implements NormalizedNodeDataInput
 
     private final DataInput input;
 
-    private final Map<Integer, String> codedStringMap = new HashMap<>();
+    private final List<String> codedStringMap = new ArrayList<>();
 
     private QName lastLeafSetQName;
 
@@ -260,14 +260,14 @@ public class NormalizedNodeInputStreamReader implements NormalizedNodeDataInput
                 return null;
             case TokenTypes.IS_CODE_VALUE:
                 final int code = input.readInt();
-                final String lookup = codedStringMap.get(code);
-                if (lookup == null) {
-                    throw new IOException("String code " + code + " was not found");
+                try {
+                    return codedStringMap.get(code);
+                } catch (IndexOutOfBoundsException e) {
+                    throw new IOException("String code " + code + " was not found", e);
                 }
-                return lookup;
             case TokenTypes.IS_STRING_VALUE:
                 final String value = input.readUTF().intern();
-                codedStringMap.put(codedStringMap.size(), value);
+                codedStringMap.add(value);
                 return value;
             default:
                 throw new IOException("Unhandled string value type " + valueType);

©2013 OpenDaylight, A Linux Foundation Collaborative Project. All Rights Reserved.
OpenDaylight is a registered trademark of The OpenDaylight Project, Inc.
Linux Foundation and OpenDaylight are registered trademarks of the Linux Foundation.
Linux is a registered trademark of Linus Torvalds.