You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@kylin.apache.org by GitBox <gi...@apache.org> on 2021/07/28 10:26:45 UTC

[GitHub] [kylin] lgtm-com[bot] commented on pull request #1702: KYLIN-5052 Clean up unused and out-of-date source code/config

lgtm-com[bot] commented on pull request #1702:
URL: https://github.com/apache/kylin/pull/1702#issuecomment-888199035


   This pull request **fixes 14 alerts** when merging 5bd8f58710038764aacdced2a12e40ad0f8348e1 into 9f7e5316effee916605cb883f9accfa46b849071 - [view on LGTM.com](https://lgtm.com/projects/g/apache/kylin/rev/pr-98da95ede19baf3f378b211788d3c57dba6d5adf)
   
   **fixed alerts:**
   
   * 3 for Boxed variable is never null
   * 2 for Useless comparison test
   * 2 for Query built from user\-controlled sources
   * 2 for Dereferenced variable may be null
   * 2 for Unused format argument
   * 1 for Result of multiplication cast to wider type
   * 1 for Implicit narrowing conversion in compound assignment
   * 1 for Potential input resource leak


-- 
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: issues-unsubscribe@kylin.apache.org

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