Remove accidentally duplicated SpanKey

Signed-off-by: Bob Weinand <[email protected]>

Author: bwoebi

datadog-trace-utils/src/msgpack_decoder/v04/span.rs

@@ -10,7 +10,7 @@ use crate::msgpack_decoder::decode::string::{
     read_nullable_str_map_to_strings, read_nullable_string,
 };
 use crate::msgpack_decoder::decode::{meta_struct::read_meta_struct, metrics::read_metrics};
-use crate::span::{v04::Span, SpanKey, TraceData};
+use crate::span::{v04::Span, v04::SpanKey, TraceData};
 use std::borrow::Borrow;
 
 /// Decodes a slice of bytes into a `Span` object.

datadog-trace-utils/src/span/mod.rs

@@ -17,50 +17,6 @@ use std::marker::PhantomData;
 use std::str::FromStr;
 use tinybytes::{Bytes, BytesString};
 
-#[derive(Debug, PartialEq)]
-pub enum SpanKey {
-    Service,
-    Name,
-    Resource,
-    TraceId,
-    SpanId,
-    ParentId,
-    Start,
-    Duration,
-    Error,
-    Meta,
-    Metrics,
-    Type,
-    MetaStruct,
-    SpanLinks,
-    SpanEvents,
-}
-
-impl FromStr for SpanKey {
-    type Err = SpanKeyParseError;
-
-    fn from_str(s: &str) -> Result<Self, Self::Err> {
-        match s {
-            "service" => Ok(SpanKey::Service),
-            "name" => Ok(SpanKey::Name),
-            "resource" => Ok(SpanKey::Resource),
-            "trace_id" => Ok(SpanKey::TraceId),
-            "span_id" => Ok(SpanKey::SpanId),
-            "parent_id" => Ok(SpanKey::ParentId),
-            "start" => Ok(SpanKey::Start),
-            "duration" => Ok(SpanKey::Duration),
-            "error" => Ok(SpanKey::Error),
-            "meta" => Ok(SpanKey::Meta),
-            "metrics" => Ok(SpanKey::Metrics),
-            "type" => Ok(SpanKey::Type),
-            "meta_struct" => Ok(SpanKey::MetaStruct),
-            "span_links" => Ok(SpanKey::SpanLinks),
-            "span_events" => Ok(SpanKey::SpanEvents),
-            _ => Err(SpanKeyParseError::new(format!("Invalid span key: {s}"))),
-        }
-    }
-}
-
 /// Trait representing the requirements for a type to be used as a Span "string" type.
 /// Note: Borrow<str> is not required by the derived traits, but allows to access HashMap elements
 /// from a static str and check if the string is empty.