Remove potentially confusing String key override for Value<?>

Author: trask

api/incubator/src/main/java/io/opentelemetry/api/incubator/common/ExtendedAttributesBuilder.java

@@ -14,7 +14,6 @@
 import static io.opentelemetry.api.incubator.common.ExtendedAttributeKey.longKey;
 import static io.opentelemetry.api.incubator.common.ExtendedAttributeKey.stringArrayKey;
 import static io.opentelemetry.api.incubator.common.ExtendedAttributeKey.stringKey;
-import static io.opentelemetry.api.incubator.common.ExtendedAttributeKey.valueKey;
 
 import io.opentelemetry.api.common.AttributeKey;
 import io.opentelemetry.api.common.Attributes;
@@ -93,7 +92,8 @@ default ExtendedAttributesBuilder put(String key, boolean value) {
    * <p>Note: It is strongly recommended to use {@link #put(ExtendedAttributeKey, Object)}, and
    * pre-allocate your keys, if possible.
    *
-   * @deprecated Use {@link #put(String, Value)} with {@link Value#of(java.util.Map)} instead.
+   * @deprecated Use {@link #put(ExtendedAttributeKey, Object)} with {@link
+   *     ExtendedAttributeKey#valueKey(String)} and {@link Value#of(java.util.Map)} instead.
    * @return this Builder
    */
   @Deprecated
@@ -102,18 +102,6 @@ default <T> ExtendedAttributesBuilder put(String key, ExtendedAttributes value)
     return put(ExtendedAttributeKey.extendedAttributesKey(key), value);
   }
 
-  /**
-   * Puts a {@link Value} attribute into this.
-   *
-   * <p>Note: It is strongly recommended to use {@link #put(ExtendedAttributeKey, Object)}, and
-   * pre-allocate your keys, if possible.
-   *
-   * @return this Builder
-   */
-  default ExtendedAttributesBuilder put(String key, Value<?> value) {
-    return put(valueKey(key), value);
-  }
-
   /**
    * Puts a String array attribute into this.
    *

api/incubator/src/test/java/io/opentelemetry/api/incubator/common/ExtendedAttributesTest.java

@@ -217,9 +217,6 @@ private static Stream<Arguments> attributesArgs() {
             ImmutableMap.builder()
                 .put("key", ImmutableMap.builder().put("child", "value").build())
                 .build()),
-        Arguments.of(
-            ExtendedAttributes.builder().put("key", Value.of("value")).build(),
-            ImmutableMap.builder().put("key", Value.of("value")).build()),
         Arguments.of(
             ExtendedAttributes.builder()
                 .put(ExtendedAttributeKey.stringKey("key"), "value")
@@ -281,7 +278,7 @@ private static Stream<Arguments> attributesArgs() {
                 .put("key8", 1L, 2L)
                 .put("key9", 1.1, 2.2)
                 .put("key10", ExtendedAttributes.builder().put("child", "value").build())
-                .put("key11", Value.of("value"))
+                .put(ExtendedAttributeKey.valueKey("key11"), Value.of("value"))
                 .build(),
             ImmutableMap.builder()
                 .put("key1", "value1")