diff --git a/src/main/java/com/profiler/sender/LoggingDataSender.java b/src/main/java/com/profiler/sender/LoggingDataSender.java index 3a9afca7b..00d546ab7 100644 --- a/src/main/java/com/profiler/sender/LoggingDataSender.java +++ b/src/main/java/com/profiler/sender/LoggingDataSender.java @@ -16,13 +16,13 @@ public class LoggingDataSender implements DataSender { @Override public boolean send(TBase data) { - logger.info("send tBase:" + data); + logger.info("send tBase:{]", data); return true; } @Override public boolean send(Thriftable thriftable) { - logger.info("send thriftable:" + thriftable); + logger.info("send thriftable:{}", thriftable); return true; //To change body of implemented methods use File | Settings | File Templates. } diff --git a/src/main/java/com/profiler/sender/UdpDataSender.java b/src/main/java/com/profiler/sender/UdpDataSender.java index bbf0b81b7..51265b043 100644 --- a/src/main/java/com/profiler/sender/UdpDataSender.java +++ b/src/main/java/com/profiler/sender/UdpDataSender.java @@ -200,7 +200,7 @@ public class UdpDataSender implements DataSender, Runnable { } else if (dto instanceof Thriftable) { tBase = ((Thriftable) dto).toThrift(); } else { - logger.warn("sendPacket fail. invalid type:" + dto.getClass()); + logger.warn("sendPacket fail. invalid type:{}", dto.getClass()); return; } // single thread이므로 데이터 array를 nocopy해서 보냄. @@ -215,10 +215,10 @@ public class UdpDataSender implements DataSender, Runnable { try { udpSocket.send(reusePacket); if (logger.isInfoEnabled()) { - logger.info("Data sent. " + dto); + logger.info("Data sent. {}", dto); } } catch (IOException e) { - logger.warn("packet send error " + dto, e); + logger.warn("packet send error {}", dto, e); } } @@ -253,7 +253,7 @@ public class UdpDataSender implements DataSender, Runnable { return serializer.serialize(header, dto); } catch (TException e) { if (logger.isWarnEnabled()) { - logger.warn("Serialize fail:" + dto + " Caused:" + e.getMessage(), e); + logger.warn("Serialize fail:{} Caused:{}", new Object[] { dto, e.getMessage(), e}); } return null; }