Avoid costly metrics collection when metric collector is NoOpMetricCollector

Author: roudech1

.changes/next-release/bugfix-AWSSDKforJavav2-3b17ee3.json

@@ -0,0 +1,6 @@
+{
+    "category": "AWS SDK for Java v2", 
+    "contributor": "@croudet", 
+    "type": "bugfix", 
+    "description": "Avoid costly metrics collection when metric collector is NoOpMetricCollector."
+}

core/sdk-core/src/main/java/software/amazon/awssdk/core/internal/util/MetricUtils.java

@@ -67,7 +67,7 @@ public static <T> Pair<T, Duration> measureDurationUnsafe(Callable<T> c) throws
     }
 
     public static void collectHttpMetrics(MetricCollector metricCollector, SdkHttpFullResponse httpResponse) {
-        if (metricCollector != null && httpResponse != null) {
+        if (metricCollector != null && !(metricCollector instanceof NoOpMetricCollector) && httpResponse != null) {
             metricCollector.reportMetric(HttpMetric.HTTP_STATUS_CODE, httpResponse.statusCode());
             SdkHttpUtils.allMatchingHeadersFromCollection(httpResponse.headers(), X_AMZN_REQUEST_ID_HEADERS)
                         .forEach(v -> metricCollector.reportMetric(CoreMetric.AWS_REQUEST_ID, v));