Fix GraphiteSender retry log message

Author: rbygrave

metrics-graphite/src/main/java/io/avaje/metrics/graphite/DGraphiteSender.java

@@ -148,7 +148,7 @@ public void close() {
     try {
       flush();
     } catch (IOException e) {
-      log.log(INFO, "Exception flushing metrics {0}", e);
+      log.log(INFO, "Exception flushing metrics", e);
     } finally {
       closeSocket();
     }
@@ -159,26 +159,20 @@ private void closeSocket() {
       try {
         socket.close();
       } catch (IOException e) {
-        log.log(INFO, "Exception trying to close socket {0}", e);
+        log.log(INFO, "Exception trying to close socket", e);
       }
       socket = null;
     }
   }
 
-  /**
-   * 1. Run the pickler script to package all the pending metrics into a single message
-   * 2. Send the message to graphite
-   * 3. Clear out the list of metrics
-   */
   private void writeMetrics() throws IOException {
     if (!metrics.isEmpty()) {
       try {
         final byte[] payload = pickleMetrics(metrics);
         final byte[] header = ByteBuffer.allocate(4).putInt(payload.length).array();
         send(header, payload);
       } finally {
-        // if there was an error, we might miss some data. for now, drop those on the floor and
-        // try to keep going.
+        // if there was an error we might drop some metrics
         metrics.clear();
       }
     }
@@ -188,7 +182,8 @@ private void send(byte[] header, byte[] payload) throws IOException {
     try {
       sendPayload(header, payload);
     } catch (IOException e) {
-      log.log(WARNING, "Retry sending metrics due to {0}", e);
+      // perform a single retry with a new socket connection
+      log.log(WARNING, "Retry sending metrics due to " + e);
       closeSocket();
       this.socket = socketFactory.createSocket(address.getAddress(), address.getPort());
       sendPayload(header, payload);