[#noissue] cleanup duplicate logs

pinpoint-apm · Jun 25, 2019 · 27142d1 · 27142d1
1 parent 8a3cd33
commit 27142d1
Show file tree

Hide file tree

Showing 8 changed files with 1 addition and 34 deletions.
diff --git a/...rc/main/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftAgentEventHandler.java b/...rc/main/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftAgentEventHandler.java
@@ -58,7 +58,7 @@ public class ThriftAgentEventHandler implements SimpleHandler {
     public void handleSimple(ServerRequest serverRequest) {
         final Object data = serverRequest.getData();
         if (logger.isDebugEnabled()) {
-            logger.debug("Handle simple={}", data);
+            logger.debug("Handle simple data={}", data);
         }
 
         if (data instanceof TAgentStat) {
@@ -71,9 +71,6 @@ public void handleSimple(ServerRequest serverRequest) {
     }
 
     private void handleAgentStat(TAgentStat agentStat) {
-        if (logger.isDebugEnabled()) {
-            logger.debug("Handle TAgentStat={}", agentStat);
-        }
         final AgentEventBo agentEventBo = this.agentEventMapper.map(agentStat);
         if (agentEventBo == null) {
             return;
@@ -82,9 +79,6 @@ private void handleAgentStat(TAgentStat agentStat) {
     }
 
     private void handleAgentStatBatch(TAgentStatBatch tAgentStatBatch) {
-        if (logger.isDebugEnabled()) {
-            logger.debug("Handle TAgentStatBatch={}", tAgentStatBatch);
-        }
         final List<AgentEventBo> agentEventBoList = this.agentEventBatchMapper.map(tAgentStatBatch);
         if (CollectionUtils.isEmpty(agentEventBoList)) {
             return;

diff --git a/...src/main/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftAgentInfoHandler.java b/...src/main/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftAgentInfoHandler.java
@@ -75,9 +75,6 @@ public void handleRequest(ServerRequest serverRequest, ServerResponse serverResp
     }
 
     private Object handleAgentInfo(TAgentInfo agentInfo) {
-        if (logger.isDebugEnabled()) {
-            logger.debug("Handle TAgentInfo={}", agentInfo);
-        }
         try {
             // agent info
             final AgentInfoBo agentInfoBo = this.agentInfoBoMapper.map(agentInfo);

diff --git a/...c/main/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftAgentStatHandlerV2.java b/...c/main/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftAgentStatHandlerV2.java
@@ -80,9 +80,6 @@ void handleSimple(TBase<?, ?> tBase) {
     }
 
     private void handleAgentStat(TAgentStat tAgentStat) {
-        if (logger.isDebugEnabled()) {
-            logger.debug("Handle TAgentStat={}", tAgentStat);
-        }
         final AgentStatBo agentStatBo = this.agentStatMapper.map(tAgentStat);
         if (agentStatBo == null) {
             return;
@@ -94,9 +91,6 @@ private void handleAgentStat(TAgentStat tAgentStat) {
     }
 
     private void handleAgentStatBatch(TAgentStatBatch tAgentStatBatch) {
-        if (logger.isDebugEnabled()) {
-            logger.debug("Handle TAgentStatBatch={}", tAgentStatBatch);
-        }
         final AgentStatBo agentStatBo = this.agentStatBatchMapper.map(tAgentStatBatch);
         if (agentStatBo == null) {
             return;

diff --git a/...c/main/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftApiMetaDataHandler.java b/...c/main/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftApiMetaDataHandler.java
@@ -57,10 +57,6 @@ public void handleRequest(ServerRequest serverRequest, ServerResponse serverResp
     }
 
     private Object handleApiMetaData(TApiMetaData apiMetaData) {
-        if (logger.isDebugEnabled()) {
-            logger.debug("Handle TApiMetaData={}", apiMetaData);
-        }
-
         try {
             final ApiMetaDataBo apiMetaDataBo = new ApiMetaDataBo(apiMetaData.getAgentId(), apiMetaData.getAgentStartTime(), apiMetaData.getApiId());
             apiMetaDataBo.setApiInfo(apiMetaData.getApiInfo());

diff --git a/...src/main/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftSpanChunkHandler.java b/...src/main/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftSpanChunkHandler.java
@@ -58,10 +58,6 @@ public void handleSimple(ServerRequest serverRequest) {
     }
 
     private void handleSpanChunk(TSpanChunk tbase) {
-        if (logger.isDebugEnabled()) {
-            logger.debug("Handle TSpanChunk={}", tbase);
-        }
-
         try {
             final SpanChunkBo spanChunkBo = this.spanFactory.buildSpanChunkBo(tbase);
             this.traceService.insertSpanChunk(spanChunkBo);

diff --git a/...ctor/src/main/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftSpanHandler.java b/...ctor/src/main/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftSpanHandler.java
@@ -64,9 +64,6 @@ public void handleSimple(ServerRequest serverRequest) {
     }
 
     private void handleSpan(TSpan tSpan) {
-        if (logger.isDebugEnabled()) {
-            logger.debug("Handle TSpan={}", tSpan);
-        }
         try {
             final SpanBo spanBo = spanFactory.buildSpanBo(tSpan);
             traceService.insertSpan(spanBo);

diff --git a/...c/main/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftSqlMetaDataHandler.java b/...c/main/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftSqlMetaDataHandler.java
@@ -57,10 +57,6 @@ public void handleRequest(ServerRequest serverRequest, ServerResponse serverResp
     }
 
     private Object handleSqlMetaData(TSqlMetaData sqlMetaData) {
-        if (logger.isDebugEnabled()) {
-            logger.debug("Handle TSqlMetaData={}", sqlMetaData);
-        }
-
         try {
             final SqlMetaDataBo sqlMetaDataBo = new SqlMetaDataBo(sqlMetaData.getAgentId(), sqlMetaData.getAgentStartTime(), sqlMetaData.getSqlId());
             sqlMetaDataBo.setSql(sqlMetaData.getSql());

diff --git a/...ain/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftStringMetaDataHandler.java b/...ain/java/com/navercorp/pinpoint/collector/handler/thrift/ThriftStringMetaDataHandler.java
@@ -56,9 +56,6 @@ public void handleRequest(ServerRequest serverRequest, ServerResponse serverResp
     }
 
     private Object handleStringMetaData(TStringMetaData stringMetaData) {
-        if (logger.isDebugEnabled()) {
-            logger.debug("Handle TStringMetaData={}", stringMetaData);
-        }
         try {
             final StringMetaDataBo stringMetaDataBo = new StringMetaDataBo(stringMetaData.getAgentId(), stringMetaData.getAgentStartTime(), stringMetaData.getStringId());
             stringMetaDataBo.setStringValue(stringMetaData.getStringValue());