Split out BaseTypes 67/86067/2
authorRobert Varga <robert.varga@pantheon.tech>
Fri, 29 Nov 2019 13:38:06 +0000 (14:38 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Fri, 29 Nov 2019 15:52:27 +0000 (16:52 +0100)
This splits out and hides concrete BaseType classes, hiding them
and untangling BaseType.fromJson() instantiation. This makes object
lifecycle more clear, allowing for future optimization.

Change-Id: I60630b3cd7b02d6e9c241d6e666550ae041c96f9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/BaseType.java
library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/BooleanBaseType.java [new file with mode: 0644]
library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/IntegerBaseType.java [new file with mode: 0644]
library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/RealBaseType.java [new file with mode: 0644]
library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/StringBaseType.java [new file with mode: 0644]
library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/TableSchema.java
library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/UuidBaseType.java [new file with mode: 0644]

index 5f89931cbdf53e8bdef31a1f8e6f6771ee3e6b1e..8024a036204f34dec87609f350a075562f536408 100644 (file)
  * 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.ovsdb.lib.schema;
 
 import com.fasterxml.jackson.databind.JsonNode;
-import java.util.HashSet;
-import java.util.Optional;
-import java.util.Set;
 import org.opendaylight.ovsdb.lib.error.TyperException;
-import org.opendaylight.ovsdb.lib.notation.ReferencedRow;
-import org.opendaylight.ovsdb.lib.notation.UUID;
+import org.slf4j.Logger;
 
 public abstract class BaseType<E extends BaseType<E>> {
+    private static final Logger LOG = org.slf4j.LoggerFactory.getLogger(BaseType.class);
 
-    private static BaseType[] types = new BaseType[] {
-        new StringBaseType(),
-        new IntegerBaseType(),
-        new RealBaseType(),
-        new BooleanBaseType(),
-        new UuidBaseType(),
-    };
-
-    public static BaseType fromJson(JsonNode json, String keyorval) {
-        BaseType baseType = null;
+    public static BaseType fromJson(final JsonNode json, final String keyorval) {
         if (json.isValueNode()) {
-            for (BaseType baseTypeFactory : types) {
-                String type = json.asText().trim();
-                baseType = baseTypeFactory.fromString(type);
-                if (baseType != null) {
-                    break;
-                }
-            }
-        } else {
-            if (!json.has(keyorval)) {
-                throw new TyperException("Not a type");
-            }
-
-            for (BaseType baseTypeFactory : types) {
-                baseType = baseTypeFactory.fromJsonNode(json.get(keyorval), keyorval);
-                if (baseType != null) {
-                    break;
-                }
-            }
-        }
-        return baseType;
-    }
-
-    protected abstract E fromString(String type);
-
-    protected abstract void getConstraints(E baseType, JsonNode type);
-
-    protected E fromJsonNode(JsonNode type, String keyorval) {
-
-        E baseType = null;
-
-        //json like  "string"
-        if (type.isTextual()) {
-            baseType = fromString(type.asText());
-            if (baseType != null) {
-                return baseType;
-            }
-        }
-
-        //json like  {"type" : "string", "enum": ["set", ["access", "native-tagged"]]}" for key or value
-        if (type.isObject() && type.has("type")) {
-            baseType = fromString(type.get("type").asText());
-            if (baseType != null) {
-                getConstraints(baseType, type);
-            }
-        }
-
-        return baseType;
-    }
-
-    public abstract Object toValue(JsonNode value);
-
-    public abstract void validate(Object value);
-
-    public static class IntegerBaseType extends BaseType<IntegerBaseType> {
-        long min = Long.MIN_VALUE;
-        long max = Long.MAX_VALUE;
-        Set<Integer> enums;
-
-        @Override
-        public IntegerBaseType fromString(String typeString) {
-            return "integer".equals(typeString) ? new IntegerBaseType() : null;
-        }
-
-        @Override
-        protected void getConstraints(IntegerBaseType baseType, JsonNode type) {
-            JsonNode typeMaxNode = type.get("maxInteger");
-            if (typeMaxNode != null) {
-                baseType.setMax(typeMaxNode.asLong());
-            }
-            JsonNode typeMinNode = type.get("minInteger");
-            if (typeMinNode != null) {
-                baseType.setMin(typeMinNode.asLong());
-            }
-            Optional<Set<Integer>> typeEnumsOpt = populateEnum(type);
-            if (typeEnumsOpt.isPresent()) {
-                baseType.setEnums(typeEnumsOpt.get());
-            }
-        }
-
-        @Override
-        public Object toValue(JsonNode value) {
-            return value.asLong();
-        }
-
-        @Override
-        public void validate(Object value) {
-
-        }
-
-        private Optional<Set<Integer>> populateEnum(JsonNode node) {
-            if (node.has("enum")) {
-                Set<Integer> nodesEnums = new HashSet<>();
-                JsonNode anEnum = node.get("enum").get(1);
-                for (JsonNode enm : anEnum) {
-                    nodesEnums.add(enm.asInt());
-                }
-                return Optional.of(nodesEnums);
-            } else {
-                return Optional.empty();
-            }
-        }
-
-
-        public long getMin() {
-            return min;
-        }
-
-        public void setMin(long min) {
-            this.min = min;
-        }
-
-        public long getMax() {
-            return max;
-        }
-
-        public void setMax(long max) {
-            this.max = max;
-        }
-
-        public Set<Integer> getEnums() {
-            return enums;
-        }
-
-        public void setEnums(Set<Integer> enums) {
-            this.enums = enums;
-        }
-
-        @Override
-        public String toString() {
-            return "IntegerBaseType";
-        }
-
-        @Override
-        public int hashCode() {
-            final int prime = 31;
-            int result = 1;
-            result = prime * result + (enums == null ? 0 : enums.hashCode());
-            result = prime * result + (int) (max ^ max >>> 32);
-            result = prime * result + (int) (min ^ min >>> 32);
-            return result;
-        }
-
-        @Override
-        public boolean equals(Object obj) {
-            if (this == obj) {
-                return true;
-            }
-            if (obj == null) {
-                return false;
-            }
-            if (getClass() != obj.getClass()) {
-                return false;
-            }
-            IntegerBaseType other = (IntegerBaseType) obj;
-            if (enums == null) {
-                if (other.enums != null) {
-                    return false;
-                }
-            } else if (!enums.equals(other.enums)) {
-                return false;
-            }
-            if (max != other.max) {
-                return false;
-            }
-            if (min != other.min) {
-                return false;
-            }
-            return true;
-        }
-    }
-
-    public static class RealBaseType extends BaseType<RealBaseType> {
-        double min = Double.MIN_VALUE;
-        double max = Double.MAX_VALUE;
-        Set<Double> enums;
-
-        @Override
-        public RealBaseType fromString(String typeString) {
-            return "real".equals(typeString) ? new RealBaseType() : null;
-        }
-
-        @Override
-        protected void getConstraints(RealBaseType baseType, JsonNode type) {
-            JsonNode typeMaxNode = type.get("maxReal");
-            if (typeMaxNode != null) {
-                baseType.setMax(typeMaxNode.asLong());
-            }
-            JsonNode typeMinNode = type.get("minReal");
-            if (typeMinNode != null) {
-                baseType.setMin(typeMinNode.asLong());
-            }
-            Optional<Set<Double>> typeEnumsOpt = populateEnum(type);
-            if (typeEnumsOpt.isPresent()) {
-                baseType.setEnums(typeEnumsOpt.get());
-            }
-        }
-
-        @Override
-        public Object toValue(JsonNode value) {
-            return value.asDouble();
-        }
-
-        @Override
-        public void validate(Object value) {
-
-        }
-
-        private Optional<Set<Double>> populateEnum(JsonNode node) {
-            if (node.has("enum")) {
-                Set<Double> nodesEnums = new HashSet<>();
-                JsonNode anEnum = node.get("enum").get(1);
-                for (JsonNode enm : anEnum) {
-                    nodesEnums.add(enm.asDouble());
-                }
-                return Optional.of(nodesEnums);
-            } else {
-                return Optional.empty();
-            }
-        }
-
-        public double getMin() {
-            return min;
-        }
-
-        public void setMin(double min) {
-            this.min = min;
-        }
-
-        public double getMax() {
-            return max;
-        }
-
-        public void setMax(double max) {
-            this.max = max;
-        }
-
-        public Set<Double> getEnums() {
-            return enums;
-        }
-
-        public void setEnums(Set<Double> enums) {
-            this.enums = enums;
-        }
-
-        @Override
-        public String toString() {
-            return "RealBaseType";
+            return fromString(json.asText().trim());
         }
 
-        @Override
-        public int hashCode() {
-            final int prime = 31;
-            int result = 1;
-            result = prime * result + (enums == null ? 0 : enums.hashCode());
-            long temp;
-            temp = Double.doubleToLongBits(max);
-            result = prime * result + (int) (temp ^ temp >>> 32);
-            temp = Double.doubleToLongBits(min);
-            result = prime * result + (int) (temp ^ temp >>> 32);
-            return result;
+        final JsonNode type = json.get(keyorval);
+        if (type == null) {
+            throw new TyperException("Not a type");
         }
-
-        @Override
-        public boolean equals(Object obj) {
-            if (this == obj) {
-                return true;
-            }
-            if (obj == null) {
-                return false;
-            }
-            if (getClass() != obj.getClass()) {
-                return false;
-            }
-            RealBaseType other = (RealBaseType) obj;
-            if (enums == null) {
-                if (other.enums != null) {
-                    return false;
+        if (type.isTextual()) {
+            //json like  "string"
+            return fromString(type.asText());
+        }
+        if (type.isObject()) {
+            //json like  {"type" : "string", "enum": ["set", ["access", "native-tagged"]]}" for key or value
+            final JsonNode nestedType = type.get("type");
+            if (nestedType != null) {
+                final BaseType ret = fromString(nestedType.asText());
+                if (ret != null) {
+                    ret.fillConstraints(type);
+                    return ret;
                 }
-            } else if (!enums.equals(other.enums)) {
-                return false;
-            }
-            if (Double.doubleToLongBits(max) != Double.doubleToLongBits(other.max)) {
-                return false;
-            }
-            if (Double.doubleToLongBits(min) != Double.doubleToLongBits(other.min)) {
-                return false;
             }
-            return true;
         }
-    }
-
-
-    public static class BooleanBaseType extends BaseType {
 
-        @Override
-        public BooleanBaseType fromString(String typeString) {
-            return "boolean".equals(typeString) ? new BooleanBaseType() : null;
-        }
-
-        @Override
-        protected void getConstraints(BaseType baseType, JsonNode node) {
-            //no op
-        }
-
-        @Override
-        public Object toValue(JsonNode value) {
-            return value.asBoolean();
-        }
-
-        @Override
-        public void validate(Object value) {
-
-        }
-
-        @Override
-        public String toString() {
-            return "BooleanBaseType";
-        }
+        return null;
     }
 
-    public static class StringBaseType extends BaseType<StringBaseType> {
-        int minLength = Integer.MIN_VALUE;
-        int maxLength = Integer.MAX_VALUE;
-        Set<String> enums;
-
-        @Override
-        public StringBaseType fromString(String typeString) {
-            return "string".equals(typeString) ? new StringBaseType() : null;
-        }
-
-        @Override
-        protected void getConstraints(StringBaseType baseType, JsonNode type) {
-            JsonNode typeMaxNode = type.get("maxLength");
-            if (typeMaxNode != null) {
-                baseType.setMaxLength(typeMaxNode.asInt());
-            }
-            JsonNode typeMinNode = type.get("minLength");
-            if (typeMinNode != null) {
-                baseType.setMinLength(typeMinNode.asInt());
-            }
-            Optional<Set<String>> typeEnumsOpt = populateEnum(type);
-            if (typeEnumsOpt.isPresent()) {
-                baseType.setEnums(typeEnumsOpt.get());
-            }
-        }
-
-        @Override
-        public Object toValue(JsonNode value) {
-            return value.asText();
-        }
-
-        @Override
-        public void validate(Object value) {
-
-        }
-
-        private Optional<Set<String>> populateEnum(JsonNode node) {
-            if (node.has("enum")) {
-                Set<String> nodesEnums = new HashSet<>();
-                JsonNode enumVal = node.get("enum");
-                if (enumVal.isArray()) {
-                    JsonNode anEnum = enumVal.get(1);
-                    for (JsonNode enm : anEnum) {
-                        nodesEnums.add(enm.asText());
-                    }
-                } else if (enumVal.isTextual()) {
-                    nodesEnums.add(enumVal.asText());
-                }
-                return Optional.of(nodesEnums);
-            } else {
-                return Optional.empty();
-            }
-        }
-
-        public int getMinLength() {
-            return minLength;
-        }
-
-        public void setMinLength(int minLength) {
-            this.minLength = minLength;
-        }
-
-        public int getMaxLength() {
-            return maxLength;
-        }
-
-        public void setMaxLength(int maxLength) {
-            this.maxLength = maxLength;
-        }
-
-        public Set<String> getEnums() {
-            return enums;
-        }
-
-        public void setEnums(Set<String> enums) {
-            this.enums = enums;
-        }
+    abstract void fillConstraints(JsonNode type);
 
-        @Override
-        public String toString() {
-            return "StringBaseType";
-        }
+    public abstract Object toValue(JsonNode value);
 
-        @Override
-        public int hashCode() {
-            final int prime = 31;
-            int result = 1;
-            result = prime * result + (enums == null ? 0 : enums.hashCode());
-            result = prime * result + maxLength;
-            result = prime * result + minLength;
-            return result;
-        }
+    public abstract void validate(Object value);
 
-        @Override
-        public boolean equals(Object obj) {
-            if (this == obj) {
-                return true;
-            }
-            if (obj == null) {
-                return false;
-            }
-            if (getClass() != obj.getClass()) {
-                return false;
-            }
-            StringBaseType other = (StringBaseType) obj;
-            if (enums == null) {
-                if (other.enums != null) {
-                    return false;
-                }
-            } else if (!enums.equals(other.enums)) {
-                return false;
-            }
-            if (maxLength != other.maxLength) {
-                return false;
-            }
-            if (minLength != other.minLength) {
-                return false;
-            }
-            return true;
+    private static BaseType fromString(final String type) {
+        switch (type) {
+            case "boolean":
+                return new BooleanBaseType();
+            case "integer":
+                return new IntegerBaseType();
+            case "real":
+                return new RealBaseType();
+            case "string":
+                return new StringBaseType();
+            case "uuid":
+                return new UuidBaseType();
+            default:
+                LOG.debug("Unknown base type {}", type);
+                return null;
         }
-
     }
 
-
-    public static class UuidBaseType extends BaseType<UuidBaseType> {
-        // These enum types correspond to JSON values and need to be in lower-case currently
-        public enum RefType { strong, weak }
-
-        String refTable;
-        RefType refType;
-
-        @Override
-        public UuidBaseType fromString(String typeString) {
-            return "uuid".equals(typeString) ? new UuidBaseType() : null;
-        }
-
-        @Override
-        protected void getConstraints(UuidBaseType baseType, JsonNode node) {
-            JsonNode refTableNode = node.get("refTable");
-            baseType.setRefTable(refTableNode != null ? refTableNode.asText() : null);
-
-            JsonNode refTypeJson = node.get("refType");
-            baseType.setRefType(refTypeJson != null ? RefType.valueOf(refTypeJson.asText()) : RefType.strong);
-        }
-
-        @Override
-        public Object toValue(JsonNode value) {
-            if (value.isArray()) {
-                if (value.size() == 2 && value.get(0).isTextual() && "uuid".equals(value.get(0).asText())) {
-                    return new UUID(value.get(1).asText());
-                }
-            } else {
-                /*
-                 * UUIDBaseType used by RefTable from SouthBound will always be an Array of ["uuid", <uuid>].
-                 * But there are some cases from northbound where the RefTable type can be expanded to a Row
-                 * with contents. In those scenarios, just retain the content and return a ReferencedRow for
-                 * the upper layer functions to process it.
-                 */
-                return new ReferencedRow(refTable, value);
-            }
-            return null;
-        }
-
-        @Override
-        public void validate(Object value) {
-
-        }
-
-        public String getRefTable() {
-            return refTable;
-        }
-
-        public void setRefTable(String refTable) {
-            this.refTable = refTable;
-        }
-
-        public RefType getRefType() {
-            return refType;
-        }
-
-        public void setRefType(RefType refType) {
-            this.refType = refType;
-        }
-
-        @Override
-        public String toString() {
-            return "UuidBaseType";
-        }
-
-        @Override
-        public int hashCode() {
-            final int prime = 31;
-            int result = 1;
-            result = prime * result
-                    + (refTable == null ? 0 : refTable.hashCode());
-            result = prime * result
-                    + (refType == null ? 0 : refType.hashCode());
-            return result;
-        }
-
-        @Override
-        public boolean equals(Object obj) {
-            if (this == obj) {
-                return true;
-            }
-            if (obj == null) {
-                return false;
-            }
-            if (getClass() != obj.getClass()) {
-                return false;
-            }
-            UuidBaseType other = (UuidBaseType) obj;
-            if (refTable == null) {
-                if (other.refTable != null) {
-                    return false;
-                }
-            } else if (!refTable.equals(other.refTable)) {
-                return false;
-            }
-            if (refType != other.refType) {
-                return false;
-            }
-            return true;
-        }
-    }
 }
diff --git a/library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/BooleanBaseType.java b/library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/BooleanBaseType.java
new file mode 100644 (file)
index 0000000..cef6829
--- /dev/null
@@ -0,0 +1,33 @@
+/*
+ * Copyright © 2014, 2017 EBay Software Foundation 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.ovsdb.lib.schema;
+
+import com.fasterxml.jackson.databind.JsonNode;
+
+final class BooleanBaseType extends BaseType {
+
+    @Override
+    void fillConstraints(final JsonNode node) {
+        //no op
+    }
+
+    @Override
+    public Object toValue(final JsonNode value) {
+        return value.asBoolean();
+    }
+
+    @Override
+    public void validate(final Object value) {
+
+    }
+
+    @Override
+    public String toString() {
+        return "BooleanBaseType";
+    }
+}
\ No newline at end of file
diff --git a/library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/IntegerBaseType.java b/library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/IntegerBaseType.java
new file mode 100644 (file)
index 0000000..0152d0a
--- /dev/null
@@ -0,0 +1,125 @@
+/*
+ * Copyright © 2014, 2017 EBay Software Foundation 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.ovsdb.lib.schema;
+
+import com.fasterxml.jackson.databind.JsonNode;
+import java.util.HashSet;
+import java.util.Optional;
+import java.util.Set;
+
+final class IntegerBaseType extends BaseType<IntegerBaseType> {
+    private long min = Long.MIN_VALUE;
+    private long max = Long.MAX_VALUE;
+    private Set<Integer> enums;
+
+    @Override
+    void fillConstraints(final JsonNode type) {
+        JsonNode typeMaxNode = type.get("maxInteger");
+        if (typeMaxNode != null) {
+            setMax(typeMaxNode.asLong());
+        }
+        JsonNode typeMinNode = type.get("minInteger");
+        if (typeMinNode != null) {
+            setMin(typeMinNode.asLong());
+        }
+        Optional<Set<Integer>> typeEnumsOpt = populateEnum(type);
+        if (typeEnumsOpt.isPresent()) {
+            setEnums(typeEnumsOpt.get());
+        }
+    }
+
+    @Override
+    public Object toValue(final JsonNode value) {
+        return value.asLong();
+    }
+
+    @Override
+    public void validate(final Object value) {
+
+    }
+
+    private static Optional<Set<Integer>> populateEnum(final JsonNode node) {
+        if (node.has("enum")) {
+            Set<Integer> nodesEnums = new HashSet<>();
+            JsonNode anEnum = node.get("enum").get(1);
+            for (JsonNode enm : anEnum) {
+                nodesEnums.add(enm.asInt());
+            }
+            return Optional.of(nodesEnums);
+        } else {
+            return Optional.empty();
+        }
+    }
+
+    public long getMin() {
+        return min;
+    }
+
+    public void setMin(final long min) {
+        this.min = min;
+    }
+
+    public long getMax() {
+        return max;
+    }
+
+    public void setMax(final long max) {
+        this.max = max;
+    }
+
+    public Set<Integer> getEnums() {
+        return enums;
+    }
+
+    public void setEnums(final Set<Integer> enums) {
+        this.enums = enums;
+    }
+
+    @Override
+    public String toString() {
+        return "IntegerBaseType";
+    }
+
+    @Override
+    public int hashCode() {
+        final int prime = 31;
+        int result = 1;
+        result = prime * result + (enums == null ? 0 : enums.hashCode());
+        result = prime * result + (int) (max ^ max >>> 32);
+        result = prime * result + (int) (min ^ min >>> 32);
+        return result;
+    }
+
+    @Override
+    public boolean equals(final Object obj) {
+        if (this == obj) {
+            return true;
+        }
+        if (obj == null) {
+            return false;
+        }
+        if (getClass() != obj.getClass()) {
+            return false;
+        }
+        IntegerBaseType other = (IntegerBaseType) obj;
+        if (enums == null) {
+            if (other.enums != null) {
+                return false;
+            }
+        } else if (!enums.equals(other.enums)) {
+            return false;
+        }
+        if (max != other.max) {
+            return false;
+        }
+        if (min != other.min) {
+            return false;
+        }
+        return true;
+    }
+}
\ No newline at end of file
diff --git a/library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/RealBaseType.java b/library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/RealBaseType.java
new file mode 100644 (file)
index 0000000..a3f6018
--- /dev/null
@@ -0,0 +1,128 @@
+/*
+ * Copyright © 2014, 2017 EBay Software Foundation 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.ovsdb.lib.schema;
+
+import com.fasterxml.jackson.databind.JsonNode;
+import java.util.HashSet;
+import java.util.Optional;
+import java.util.Set;
+
+final class RealBaseType extends BaseType<RealBaseType> {
+    private double min = Double.MIN_VALUE;
+    private double max = Double.MAX_VALUE;
+    private Set<Double> enums;
+
+    @Override
+    void fillConstraints(final JsonNode type) {
+        JsonNode typeMaxNode = type.get("maxReal");
+        if (typeMaxNode != null) {
+            setMax(typeMaxNode.asLong());
+        }
+        JsonNode typeMinNode = type.get("minReal");
+        if (typeMinNode != null) {
+            setMin(typeMinNode.asLong());
+        }
+        Optional<Set<Double>> typeEnumsOpt = populateEnum(type);
+        if (typeEnumsOpt.isPresent()) {
+            setEnums(typeEnumsOpt.get());
+        }
+    }
+
+    @Override
+    public Object toValue(final JsonNode value) {
+        return value.asDouble();
+    }
+
+    @Override
+    public void validate(final Object value) {
+
+    }
+
+    private static Optional<Set<Double>> populateEnum(final JsonNode node) {
+        if (node.has("enum")) {
+            Set<Double> nodesEnums = new HashSet<>();
+            JsonNode anEnum = node.get("enum").get(1);
+            for (JsonNode enm : anEnum) {
+                nodesEnums.add(enm.asDouble());
+            }
+            return Optional.of(nodesEnums);
+        } else {
+            return Optional.empty();
+        }
+    }
+
+    public double getMin() {
+        return min;
+    }
+
+    public void setMin(final double min) {
+        this.min = min;
+    }
+
+    public double getMax() {
+        return max;
+    }
+
+    public void setMax(final double max) {
+        this.max = max;
+    }
+
+    public Set<Double> getEnums() {
+        return enums;
+    }
+
+    public void setEnums(final Set<Double> enums) {
+        this.enums = enums;
+    }
+
+    @Override
+    public String toString() {
+        return "RealBaseType";
+    }
+
+    @Override
+    public int hashCode() {
+        final int prime = 31;
+        int result = 1;
+        result = prime * result + (enums == null ? 0 : enums.hashCode());
+        long temp;
+        temp = Double.doubleToLongBits(max);
+        result = prime * result + (int) (temp ^ temp >>> 32);
+        temp = Double.doubleToLongBits(min);
+        result = prime * result + (int) (temp ^ temp >>> 32);
+        return result;
+    }
+
+    @Override
+    public boolean equals(final Object obj) {
+        if (this == obj) {
+            return true;
+        }
+        if (obj == null) {
+            return false;
+        }
+        if (getClass() != obj.getClass()) {
+            return false;
+        }
+        RealBaseType other = (RealBaseType) obj;
+        if (enums == null) {
+            if (other.enums != null) {
+                return false;
+            }
+        } else if (!enums.equals(other.enums)) {
+            return false;
+        }
+        if (Double.doubleToLongBits(max) != Double.doubleToLongBits(other.max)) {
+            return false;
+        }
+        if (Double.doubleToLongBits(min) != Double.doubleToLongBits(other.min)) {
+            return false;
+        }
+        return true;
+    }
+}
\ No newline at end of file
diff --git a/library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/StringBaseType.java b/library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/StringBaseType.java
new file mode 100644 (file)
index 0000000..7e1e77d
--- /dev/null
@@ -0,0 +1,130 @@
+/*
+ * Copyright © 2014, 2017 EBay Software Foundation 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.ovsdb.lib.schema;
+
+import com.fasterxml.jackson.databind.JsonNode;
+import java.util.HashSet;
+import java.util.Optional;
+import java.util.Set;
+
+final class StringBaseType extends BaseType<StringBaseType> {
+    private int minLength = Integer.MIN_VALUE;
+    private int maxLength = Integer.MAX_VALUE;
+    private Set<String> enums;
+
+    @Override
+    void fillConstraints(final JsonNode type) {
+        JsonNode typeMaxNode = type.get("maxLength");
+        if (typeMaxNode != null) {
+            setMaxLength(typeMaxNode.asInt());
+        }
+        JsonNode typeMinNode = type.get("minLength");
+        if (typeMinNode != null) {
+            setMinLength(typeMinNode.asInt());
+        }
+        Optional<Set<String>> typeEnumsOpt = populateEnum(type);
+        if (typeEnumsOpt.isPresent()) {
+            setEnums(typeEnumsOpt.get());
+        }
+    }
+
+    @Override
+    public Object toValue(final JsonNode value) {
+        return value.asText();
+    }
+
+    @Override
+    public void validate(final Object value) {
+
+    }
+
+    private static Optional<Set<String>> populateEnum(final JsonNode node) {
+        if (node.has("enum")) {
+            Set<String> nodesEnums = new HashSet<>();
+            JsonNode enumVal = node.get("enum");
+            if (enumVal.isArray()) {
+                JsonNode anEnum = enumVal.get(1);
+                for (JsonNode enm : anEnum) {
+                    nodesEnums.add(enm.asText());
+                }
+            } else if (enumVal.isTextual()) {
+                nodesEnums.add(enumVal.asText());
+            }
+            return Optional.of(nodesEnums);
+        } else {
+            return Optional.empty();
+        }
+    }
+
+    public int getMinLength() {
+        return minLength;
+    }
+
+    public void setMinLength(final int minLength) {
+        this.minLength = minLength;
+    }
+
+    public int getMaxLength() {
+        return maxLength;
+    }
+
+    public void setMaxLength(final int maxLength) {
+        this.maxLength = maxLength;
+    }
+
+    public Set<String> getEnums() {
+        return enums;
+    }
+
+    public void setEnums(final Set<String> enums) {
+        this.enums = enums;
+    }
+
+    @Override
+    public String toString() {
+        return "StringBaseType";
+    }
+
+    @Override
+    public int hashCode() {
+        final int prime = 31;
+        int result = 1;
+        result = prime * result + (enums == null ? 0 : enums.hashCode());
+        result = prime * result + maxLength;
+        result = prime * result + minLength;
+        return result;
+    }
+
+    @Override
+    public boolean equals(final Object obj) {
+        if (this == obj) {
+            return true;
+        }
+        if (obj == null) {
+            return false;
+        }
+        if (getClass() != obj.getClass()) {
+            return false;
+        }
+        StringBaseType other = (StringBaseType) obj;
+        if (enums == null) {
+            if (other.enums != null) {
+                return false;
+            }
+        } else if (!enums.equals(other.enums)) {
+            return false;
+        }
+        if (maxLength != other.maxLength) {
+            return false;
+        }
+        if (minLength != other.minLength) {
+            return false;
+        }
+        return true;
+    }
+}
index 9ad3953d2b7d9705d2375c3a1cc2238f10a671db..ea63f412df02fd9c498f5f6b63920fb9a0a7c6a1 100644 (file)
@@ -23,7 +23,6 @@ import org.opendaylight.ovsdb.lib.notation.Column;
 import org.opendaylight.ovsdb.lib.notation.Row;
 import org.opendaylight.ovsdb.lib.notation.UUID;
 import org.opendaylight.ovsdb.lib.operations.Insert;
-import org.opendaylight.ovsdb.lib.schema.BaseType.UuidBaseType;
 
 public abstract class TableSchema<E extends TableSchema<E>> {
 
diff --git a/library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/UuidBaseType.java b/library/impl/src/main/java/org/opendaylight/ovsdb/lib/schema/UuidBaseType.java
new file mode 100644 (file)
index 0000000..7a50b76
--- /dev/null
@@ -0,0 +1,109 @@
+/*
+ * Copyright © 2014, 2017 EBay Software Foundation 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.ovsdb.lib.schema;
+
+import com.fasterxml.jackson.databind.JsonNode;
+import org.opendaylight.ovsdb.lib.notation.ReferencedRow;
+import org.opendaylight.ovsdb.lib.notation.UUID;
+
+final class UuidBaseType extends BaseType<UuidBaseType> {
+    // These enum types correspond to JSON values and need to be in lower-case currently
+    public enum RefType { strong, weak }
+
+    private String refTable;
+    private UuidBaseType.RefType refType;
+
+    @Override
+    void fillConstraints(final JsonNode node) {
+        JsonNode refTableNode = node.get("refTable");
+        setRefTable(refTableNode != null ? refTableNode.asText() : null);
+
+        JsonNode refTypeJson = node.get("refType");
+        setRefType(refTypeJson != null ? RefType.valueOf(refTypeJson.asText()) : RefType.strong);
+    }
+
+    @Override
+    public Object toValue(final JsonNode value) {
+        if (value.isArray()) {
+            if (value.size() == 2 && value.get(0).isTextual() && "uuid".equals(value.get(0).asText())) {
+                return new UUID(value.get(1).asText());
+            }
+        } else {
+            /*
+             * UUIDBaseType used by RefTable from SouthBound will always be an Array of ["uuid", <uuid>].
+             * But there are some cases from northbound where the RefTable type can be expanded to a Row
+             * with contents. In those scenarios, just retain the content and return a ReferencedRow for
+             * the upper layer functions to process it.
+             */
+            return new ReferencedRow(refTable, value);
+        }
+        return null;
+    }
+
+    @Override
+    public void validate(final Object value) {
+
+    }
+
+    public String getRefTable() {
+        return refTable;
+    }
+
+    public void setRefTable(final String refTable) {
+        this.refTable = refTable;
+    }
+
+    public UuidBaseType.RefType getRefType() {
+        return refType;
+    }
+
+    public void setRefType(final UuidBaseType.RefType refType) {
+        this.refType = refType;
+    }
+
+    @Override
+    public String toString() {
+        return "UuidBaseType";
+    }
+
+    @Override
+    public int hashCode() {
+        final int prime = 31;
+        int result = 1;
+        result = prime * result
+                + (refTable == null ? 0 : refTable.hashCode());
+        result = prime * result
+                + (refType == null ? 0 : refType.hashCode());
+        return result;
+    }
+
+    @Override
+    public boolean equals(final Object obj) {
+        if (this == obj) {
+            return true;
+        }
+        if (obj == null) {
+            return false;
+        }
+        if (getClass() != obj.getClass()) {
+            return false;
+        }
+        UuidBaseType other = (UuidBaseType) obj;
+        if (refTable == null) {
+            if (other.refTable != null) {
+                return false;
+            }
+        } else if (!refTable.equals(other.refTable)) {
+            return false;
+        }
+        if (refType != other.refType) {
+            return false;
+        }
+        return true;
+    }
+}
\ No newline at end of file