simplify

Author: trask

api/all/src/main/java/io/opentelemetry/api/common/AttributeKey.java

@@ -71,15 +71,11 @@ static AttributeKey<List<Double>> doubleArrayKey(String key) {
     return InternalAttributeKeyImpl.create(key, AttributeType.DOUBLE_ARRAY);
   }
 
-  static AttributeKey<byte[]> byteArrayKey(String key) {
-    return InternalAttributeKeyImpl.create(key, AttributeType.BYTE_ARRAY);
-  }
-
-  static AttributeKey<List<Value<?>>> valueArrayKey(String key) {
-    return InternalAttributeKeyImpl.create(key, AttributeType.VALUE_ARRAY);
-  }
-
   static AttributeKey<Attributes> mapKey(String key) {
     return InternalAttributeKeyImpl.create(key, AttributeType.MAP);
   }
+
+  static AttributeKey<Value<?>> valueKey(String key) {
+    return InternalAttributeKeyImpl.create(key, AttributeType.VALUE);
+  }
 }

api/all/src/main/java/io/opentelemetry/api/common/AttributeType.java

@@ -18,7 +18,6 @@ public enum AttributeType {
   BOOLEAN_ARRAY,
   LONG_ARRAY,
   DOUBLE_ARRAY,
-  BYTE_ARRAY,
-  VALUE_ARRAY,
-  MAP
+  MAP,
+  VALUE
 }

api/all/src/main/java/io/opentelemetry/api/common/AttributesBuilder.java

@@ -12,8 +12,10 @@
 import static io.opentelemetry.api.common.AttributeKey.doubleKey;
 import static io.opentelemetry.api.common.AttributeKey.longArrayKey;
 import static io.opentelemetry.api.common.AttributeKey.longKey;
+import static io.opentelemetry.api.common.AttributeKey.mapKey;
 import static io.opentelemetry.api.common.AttributeKey.stringArrayKey;
 import static io.opentelemetry.api.common.AttributeKey.stringKey;
+import static io.opentelemetry.api.common.AttributeKey.valueKey;
 
 import java.util.Arrays;
 import java.util.List;
@@ -164,6 +166,36 @@ default AttributesBuilder put(String key, boolean... value) {
     return put(booleanArrayKey(key), toList(value));
   }
 
+  /**
+   * Puts a nested map ({@link Attributes}) attribute into this.
+   *
+   * <p>Note: It is strongly recommended to use {@link #put(AttributeKey, Object)}, and pre-allocate
+   * your keys, if possible.
+   *
+   * @return this Builder
+   */
+  default <T> AttributesBuilder put(String key, Attributes value) {
+    if (value == null) {
+      return this;
+    }
+    return put(mapKey(key), value);
+  }
+
+  /**
+   * Puts a generic ({@link Value}) attribute into this.
+   *
+   * <p>Note: It is strongly recommended to use {@link #put(AttributeKey, Object)}, and pre-allocate
+   * your keys, if possible.
+   *
+   * @return this Builder
+   */
+  default AttributesBuilder put(String key, Value<?> value) {
+    if (value == null) {
+      return this;
+    }
+    return put(valueKey(key), value);
+  }
+
   /**
    * Puts all the provided attributes into this Builder.
    *

api/all/src/main/java/io/opentelemetry/api/common/Value.java

@@ -84,10 +84,6 @@ static Value<List<KeyValue>> of(Map<String, Value<?>> value) {
     return KeyValueList.createFromMap(value);
   }
 
-  static Value<List<KeyValue>> of(Attributes attributes) {
-    // TODO
-  }
-
   /** Returns the type of this {@link Value}. Useful for building switch statements. */
   ValueType getType();