fixed the bug of EmfMetricLoggingPublisher not properly publishing Long metrics (#6210)

* fixed the bug of not properly publishing Long metrics

* changelog added

Author: Fred1155

.changes/next-release/bugfix-AWSSDKforJavav2-17f90b1.json

@@ -0,0 +1,6 @@
+{
+    "type": "bugfix",
+    "category": "EmfMetricLoggingPublisher",
+    "contributor": "",
+    "description": "Fixed the bug that EmfMetricLoggingPublisher not properly publishing Long type metrics"
+}

metric-publishers/emf-metric-logging-publisher/src/main/java/software/amazon/awssdk/metrics/publishers/emf/internal/MetricEmfConverter.java

@@ -179,9 +179,13 @@ private void processAndWriteValue(JsonWriter jsonWriter, MetricRecord<?> mRecord
             return;
         }
 
-        if (Integer.class.isAssignableFrom(valueClass) || Long.class.isAssignableFrom(valueClass)) {
+        if (Integer.class.isAssignableFrom(valueClass)) {
             jsonWriter.writeValue((Integer) value);
         }
+
+        if (Long.class.isAssignableFrom(valueClass)) {
+            jsonWriter.writeValue((Long) value);
+        }
     }
 
     private List<String> createEmfStrings(Map<SdkMetric<?>, List<MetricRecord<?>>> aggregatedMetrics) {

metric-publishers/emf-metric-logging-publisher/src/test/java/software/amazon/awssdk/metrics/publishers/emf/internal/MetricEmfConverterTest.java

@@ -233,4 +233,21 @@ void ConvertMetricCollectionToEMF_shouldConformToSchema() throws Exception {
 
         assertThat(errors).isEmpty();
     }
+
+    @Test
+    void ConvertMetricCollectionToEMF_longValueShouldSucceed() {
+        SdkMetric<Long> metric = SdkMetric.create("TestMetric",
+                                                  Long.class,
+                                                  MetricLevel.INFO,
+                                                  MetricCategory.CUSTOM);
+
+        MetricCollector metricCollector = MetricCollector.create("test");
+        Long metricValue = 42L;
+        metricCollector.reportMetric(metric, metricValue);
+        List<String> emfLogs = metricEmfConverterDefault.convertMetricCollectionToEmf(metricCollector.collect());
+
+        assertThat(emfLogs).containsOnly("{\"_aws\":{\"Timestamp\":12345678,\"LogGroupName\":\"my_log_group_name\","
+                                         + "\"CloudWatchMetrics\":[{\"Namespace\":\"AwsSdk/JavaSdk2\",\"Dimensions\":[[]],"
+                                         + "\"Metrics\":[{\"Name\":\"TestMetric\"}]}]},\"TestMetric\":42}");
+    }
 }