Don't allocate when already valid length

Author: trask

sdk/common/src/main/java/io/opentelemetry/sdk/internal/AttributeUtil.java

@@ -156,13 +156,21 @@ private static Value<?> applyValueLengthLimit(Value<?> value, int lengthLimit) {
       return Value.of(truncated);
     } else if (type == ValueType.ARRAY) {
       List<Value<?>> array = (List<Value<?>>) value.getValue();
+      boolean allValidLength = allMatch(array, element -> isValidLengthValue(element, lengthLimit));
+      if (allValidLength) {
+        return value;
+      }
       List<Value<?>> result = new ArrayList<>(array.size());
       for (Value<?> element : array) {
         result.add(applyValueLengthLimit(element, lengthLimit));
       }
       return Value.of(result);
     } else if (type == ValueType.KEY_VALUE_LIST) {
       List<KeyValue> kvList = (List<KeyValue>) value.getValue();
+      boolean allValidLength = allMatch(kvList, kv -> isValidLengthValue(kv.getValue(), lengthLimit));
+      if (allValidLength) {
+        return value;
+      }
       List<KeyValue> result = new ArrayList<>(kvList.size());
       for (KeyValue kv : kvList) {
         result.add(KeyValue.of(kv.getKey(), applyValueLengthLimit(kv.getValue(), lengthLimit)));