feat: use new and default for ResourceBuilder

Author: pitoniak32

opentelemetry-sdk/src/resource/builder.rs

@@ -5,23 +5,25 @@ use opentelemetry::{KeyValue, Value};
 use super::{Resource, ResourceDetector, SERVICE_NAME};
 
 /// Builder to allow easy composition of a Resource
-#[derive(Debug, Default)]
+#[derive(Debug)]
 pub struct ResourceBuilder {
     resource: Resource,
 }
 
-impl ResourceBuilder {
-    /// Create ResourceBuilder with an empty [Resource].
-    pub fn new_empty() -> Self {
+impl Default for ResourceBuilder {
+    /// Create ResourceBuilder with [Resource::default()].
+    fn default() -> Self {
         ResourceBuilder {
-            resource: Resource::empty(),
+            resource: Resource::default(),
         }
     }
+}
 
-    /// Create ResourceBuilder with a default [Resource].
-    pub fn new_default() -> Self {
+impl ResourceBuilder {
+    /// Create ResourceBuilder with [Resource::empty()].
+    pub fn new() -> Self {
         ResourceBuilder {
-            resource: Resource::default(),
+            resource: Resource::empty(),
         }
     }
 

opentelemetry-sdk/src/resource/mod.rs

@@ -74,14 +74,14 @@ impl Resource {
     ///
     /// Starts with a [Resource::empty()].
     pub fn builder() -> ResourceBuilder {
-        ResourceBuilder::new_empty()
+        ResourceBuilder::new()
     }
 
     /// Creates a Builder that allows you to configure multiple aspects of the Resource.
     ///
     /// Starts with a [Resource::default()].
     pub fn builder_default() -> ResourceBuilder {
-        ResourceBuilder::new_default()
+        ResourceBuilder::default()
     }
 
     /// Creates an empty resource.