[pdata] Create real copy of Value with ValueTypeBytes type (#5267)

CHANGELOG.md

@@ -33,6 +33,7 @@
 ### 🧰 Bug fixes 🧰
 - Fix translation from otlp.Request to pdata representation, changes to the returned pdata not all reflected to the otlp.Request (#5197)
 - `exporterhelper` now properly consumes any remaining items on stop (#5203)
+- `pdata`: Fix copying of `Value` with `ValueTypeBytes` type (#5267)
 
 ## v0.49.0 Beta
 

pdata/internal/common.go

@@ -330,6 +330,14 @@ func (v Value) copyTo(dest *otlpcommon.AnyValue) {
 		}
 		// Deep copy to dest.
 		newSlice(&ov.ArrayValue.Values).CopyTo(newSlice(&av.ArrayValue.Values))
+	case *otlpcommon.AnyValue_BytesValue:
+		bv, ok := dest.Value.(*otlpcommon.AnyValue_BytesValue)
+		if !ok {
+			bv = &otlpcommon.AnyValue_BytesValue{}
+			dest.Value = bv
+		}
+		bv.BytesValue = make([]byte, len(ov.BytesValue))
+		copy(bv.BytesValue, ov.BytesValue)
 	default:
 		// Primitive immutable type, no need for deep copy.
 		dest.Value = v.orig.Value

pdata/internal/common_test.go

@@ -663,6 +663,18 @@ func TestAttributeValue_copyTo(t *testing.T) {
 	assert.EqualValues(t, nil, destVal.Value)
 }
 
+func TestValueBytes_CopyTo(t *testing.T) {
+	orig := NewValueBytes([]byte{1, 2, 3})
+	dest := NewValueEmpty()
+	orig.CopyTo(dest)
+	assert.Equal(t, orig, dest)
+
+	orig.BytesVal()[0] = 10
+	assert.NotEqual(t, orig, dest)
+	assert.Equal(t, []byte{1, 2, 3}, dest.BytesVal())
+	assert.Equal(t, []byte{10, 2, 3}, orig.BytesVal())
+}
+
 func TestMap_Update(t *testing.T) {
 	origWithNil := []otlpcommon.KeyValue{
 		{