Include stack trace when failed to send metrics to Stackdriver (#1269)

Also add successful debug logging of how many time series sent.
micrometer-metrics · Apr 3, 2019 · ac92f69 · ac92f69
1 parent ff86e92
commit ac92f69
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/...egistry-stackdriver/src/main/java/io/micrometer/stackdriver/StackdriverMeterRegistry.java b/...egistry-stackdriver/src/main/java/io/micrometer/stackdriver/StackdriverMeterRegistry.java
@@ -171,12 +171,13 @@ protected void publish() {
                         .build();
 
                 if (logger.isTraceEnabled()) {
-                    logger.trace("publishing batch to stackdriver:\n{}", request);
+                    logger.trace("publishing batch to Stackdriver:\n{}", request);
                 }
 
                 client.createTimeSeries(request);
+                logger.debug("successfully sent {} TimeSeries to Stackdriver", partition.size());
             } catch (ApiException e) {
-                logger.warn("failed to send metrics to stackdriver: {}", e.getCause().getMessage());
+                logger.warn("failed to send metrics to Stackdriver", e);
             }
         }
     }
@@ -368,7 +369,7 @@ private void createMetricDescriptorIfNecessary(MetricServiceClient client, Meter
                     client.createMetricDescriptor(request);
                     verifiedDescriptors.add(id.getName());
                 } catch (ApiException e) {
-                    logger.warn("failed to create metric descriptor in stackdriver for meter " + id + " {}", e.getCause().getMessage());
+                    logger.warn("failed to create metric descriptor in Stackdriver for meter " + id, e);
                 }
             }
         }