remove Array(Bytes)

Author: adriangb

opentelemetry-proto/src/transform/common.rs

@@ -149,7 +149,6 @@ pub mod tonic {
                         Array::I64(vals) => array_into_proto(vals),
                         Array::F64(vals) => array_into_proto(vals),
                         Array::String(vals) => array_into_proto(vals),
-                        Array::Bytes(vals) => array_into_proto(vals),
                         _ => unreachable!("Nonexistent array type"), // Needs to be updated when new array types are added
                     })),
                     _ => unreachable!("Nonexistent value type"), // Needs to be updated when new value types are added

opentelemetry/src/common.rs

@@ -189,8 +189,6 @@ pub enum Array {
     F64(Vec<f64>),
     /// Array of strings
     String(Vec<StringValue>),
-    /// Array of bytes
-    Bytes(Vec<BytesValue>),
 }
 
 impl fmt::Display for Array {
@@ -209,16 +207,6 @@ impl fmt::Display for Array {
                 }
                 write!(fmt, "]")
             }
-            Array::Bytes(values) => {
-                write!(fmt, "[")?;
-                for (i, b) in values.iter().enumerate() {
-                    if i > 0 {
-                        write!(fmt, ",")?;
-                    }
-                    write!(fmt, "{}", b)?;
-                }
-                write!(fmt, "]")
-            }
         }
     }
 }
@@ -251,7 +239,6 @@ into_array!(
     (Vec<i64>, Array::I64),
     (Vec<f64>, Array::F64),
     (Vec<StringValue>, Array::String),
-    (Vec<BytesValue>, Array::Bytes),
 );
 
 /// The value part of attribute [KeyValue] pairs.

opentelemetry/src/metrics/mod.rs

@@ -44,7 +44,6 @@ impl Hash for KeyValue {
                 Array::I64(i) => i.hash(state),
                 Array::F64(f) => f.iter().for_each(|f| F64Hashable(*f).hash(state)),
                 Array::String(s) => s.hash(state),
-                Array::Bytes(b) => b.hash(state),
             },
             Value::Bool(b) => b.hash(state),
             Value::I64(i) => i.hash(state),