You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@inlong.apache.org by GitBox <gi...@apache.org> on 2021/07/28 11:13:23 UTC

[GitHub] [incubator-inlong] dockerzhang opened a new issue #608: [INLONG-9] Remove some unnecessary code

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


   <p>1. 'String.valueOf()'</p>
   <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent">
   <pre>String strCallbackFun = req.getParameter("callback");
   if ((TStringUtils.isNotEmpty(strCallbackFun))
   && (strCallbackFun.length() <= TBaseConstants.META_MAX_CALLBACK_STRING_LENGTH)
   && (strCallbackFun.matches(TBaseConstants.META_TMP_CALLBACK_STRING_VALUE))) {
       strCallbackFun = String.valueOf(strCallbackFun).trim();
   }</pre>
   </div></div>
   <p> </p>
   
   <p>com.tencent.tubemq.server.master.web.action.screen.Webapi#execute</p>
   
   <p> </p>
   
   <p>2. {String}.substring()</p>
   <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent">
   <pre>if (path.startsWith("/")) {
       path = path.substring(1, path.length());
   }</pre>
   </div></div>
   <p>com.tencent.tubemq.server.master.web.simplemvc.RequestContext#normalizePath</p>
   
   <p> </p>
   
   <p>3.unuse init boolean</p>
   <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent">
   <pre>private boolean isOverTLS = false;
   </pre>
   </div></div>
   <p>com.tencent.tubemq.corerpc.netty.NettyRpcServer  isOverTLS </p>
   
   <p> </p>
   
   <p> </p>
   
   <p>PR <a href="https://github.com/apache/incubator-tubemq/pull/3" class="external-link" rel="nofollow">https://github.com/apache/incubator-tubemq/pull/3</a></p>
   <i>JIRA link - <a href="https://issues.apache.org/jira/browse/INLONG-9">[INLONG-9]</a> created by lan.liang</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 #608: [INLONG-9] Remove some unnecessary code

Posted by GitBox <gi...@apache.org>.
dockerzhang closed issue #608:
URL: https://github.com/apache/incubator-inlong/issues/608


   


-- 
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