MeterProvider modified to not do shutdown if user has already shut do…

…wn (#2156)

Co-authored-by: Lalit Kumar Bhasin <[email protected]>

opentelemetry-sdk/CHANGELOG.md

@@ -14,6 +14,9 @@
 - For Delta Temporality, exporters are not invoked unless there were new
   measurements since the last collect/export.
   [#2153](https://github.com/open-telemetry/opentelemetry-rust/pull/2153)
+- `MeterProvider` modified to not invoke shutdown on `Drop`, if user has already
+  called `shutdown()`.
+  [#2156](https://github.com/open-telemetry/opentelemetry-rust/pull/2156)
 
 ## v0.25.0
 

opentelemetry-sdk/src/metrics/meter_provider.rs

@@ -130,8 +130,12 @@ impl SdkMeterProviderInner {
 
 impl Drop for SdkMeterProviderInner {
     fn drop(&mut self) {
-        if let Err(err) = self.shutdown() {
-            global::handle_error(err);
+        // If user has already shutdown the provider manually by calling
+        // shutdown(), then we don't need to call shutdown again.
+        if !self.is_shutdown.load(Ordering::Relaxed) {
+            if let Err(err) = self.shutdown() {
+                global::handle_error(err);
+            }
         }
     }
 }