[GitHub] [incubator-inlong] dockerzhang opened a new issue #779: [INLONG-180] Improvement for BrokerSamplePrint

[GitBox <gi...@apache.org>]

dockerzhang opened a new issue #779:
URL: https://github.com/apache/incubator-inlong/issues/779


   <p>1. BrokerSamplePrint uses the logger transferred from the TubeBroker,  actually not necessary. We can create logger in this class.<br/>
   2. Remove the buidler in the log context</p>
   <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent">
   <pre class="code-java">
   logger.error(sBuilder
   .append(<span class="code-quote">"[heartbeat failed] heartbeat to master exception 2 is "</span>)
   .append(e.toString()).toString());
   logger.error(sBuilder
       .append(<span class="code-quote">"[heartbeat failed] heartbeat to master exception 4 is "</span>)
       .append(e.toString()).toString());
   </pre>
   </div></div>
   
   
   <i>JIRA link - <a href="https://issues.apache.org/jira/browse/INLONG-180">[INLONG-180]</a> created by technoboy</i>


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@inlong.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-inlong] dockerzhang closed issue #779: [INLONG-180] Improvement for BrokerSamplePrint

[GitBox <gi...@apache.org>]

dockerzhang closed issue #779:
URL: https://github.com/apache/incubator-inlong/issues/779


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@inlong.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org