Simplify methods

Author: utpilla

rust/otel-arrow-rust/src/pdata/mod.rs

@@ -23,15 +23,9 @@ pub mod otlp;
 #[derive(Clone, Copy, Debug)]
 pub struct TraceID([u8; 16]);
 
-type Error = &'static str;
-
-impl<'a> TraceID {
-    pub fn try_new(value: &[u8]) -> Result<TraceID, Error> {
-        if value.len() == 16 {
-            Ok(TraceID(value.try_into().unwrap()))
-        } else {
-            Err("wrong size [u8] for TraceID")
-        }
+impl TraceID {
+    pub fn new(value: &[u8; 16]) -> TraceID {
+        TraceID(*value)
     }
 }
 
@@ -50,12 +44,8 @@ impl Into<Vec<u8>> for SpanID {
     }
 }
 
-impl<'a> SpanID {
-    pub fn try_new(value: &[u8]) -> Result<SpanID, Error> {
-        if value.len() == 8 {
-            Ok(SpanID(value.try_into().unwrap()))
-        } else {
-            Err("wrong size [u8] for SpanID")
-        }
+impl SpanID {
+    pub fn new(value: &[u8; 8]) -> SpanID{
+        SpanID(*value)
     }
 }

rust/otel-arrow-rust/src/validation/testdata.rs

@@ -18,8 +18,8 @@ pub mod traces {
         let start_time = 1619712000000000000u64;
         let end_time = 1619712001000000000u64;
         let trace_id =
-            TraceID::try_new(&[1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16]).unwrap();
-        let span_id = SpanID::try_new(&[1, 2, 3, 4, 5, 6, 7, 8]).unwrap();
+            TraceID::new(&[1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16]);
+        let span_id = SpanID::new(&[1, 2, 3, 4, 5, 6, 7, 8]);
 
         let span = Span::build(trace_id, span_id, "test_span", start_time)
             .end_time_unix_nano(end_time)