Remove potentially confusing String key override for Value<?>

Author: trask

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

@@ -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")