fix: remove unnecessary thread spawning

Author: therustmonk

opentelemetry-otlp/src/exporter/http/mod.rs

@@ -127,39 +127,32 @@ impl HttpExporterBuilder {
             ))]
             {
                 // TODO - support configuring custom connector and executor
-                http_client = Some(Arc::new(HyperClient::with_default_connector(timeout, None))
-                    as Arc<dyn HttpClient>);
+                let client = HyperClient::with_default_connector(timeout, None);
+                http_client = Some(Arc::new(client));
             }
             #[cfg(all(
                 not(feature = "hyper-client"),
                 not(feature = "reqwest-blocking-client"),
                 feature = "reqwest-client"
             ))]
             {
-                http_client = Some(Arc::new(
-                    reqwest::Client::builder()
-                        .timeout(timeout)
-                        .build()
-                        .unwrap_or_default(),
-                ) as Arc<dyn HttpClient>);
+                let client = reqwest::Client::builder()
+                    .timeout(timeout)
+                    .build()
+                    .unwrap_or_default();
+                http_client = Some(Arc::new(client));
             }
             #[cfg(all(
                 not(feature = "hyper-client"),
                 not(feature = "reqwest-client"),
                 feature = "reqwest-blocking-client"
             ))]
             {
-                let timeout_clone = timeout;
-                http_client = Some(Arc::new(
-                    std::thread::spawn(move || {
-                        reqwest::blocking::Client::builder()
-                            .timeout(timeout_clone)
-                            .build()
-                            .unwrap_or_else(|_| reqwest::blocking::Client::new())
-                    })
-                    .join()
-                    .unwrap(), // TODO: Return ExporterBuildError::ThreadSpawnFailed
-                ) as Arc<dyn HttpClient>);
+                let client = reqwest::blocking::Client::builder()
+                    .timeout(timeout)
+                    .build()
+                    .unwrap_or_default();
+                http_client = Some(Arc::new(client));
             }
         }