lgtm-com[bot] commented on pull request #1327: URL: https://github.com/apache/kylin/pull/1327#issuecomment-664064157
This pull request **fixes 38 alerts** when merging 9ff4b7eca700ead4d76fac50a364d114df9b2137 into 95b1af050a708dbd16710e530a8df10335eb09fd - [view on LGTM.com](https://lgtm.com/projects/g/apache/kylin/rev/pr-079b44d1d9ec6604495b8fd708ab1f7b86a9ff95) **fixed alerts:** * 8 for Dereferenced variable may be null * 6 for Boxed variable is never null * 5 for Implicit narrowing conversion in compound assignment * 4 for Useless null check * 3 for Useless comparison test * 2 for Implicit conversion from array to string * 2 for Spurious Javadoc @param tags * 2 for Unused format argument * 2 for Potential input resource leak * 1 for Result of multiplication cast to wider type * 1 for Potential database resource leak * 1 for Potential output resource leak * 1 for Iterable wrapping an iterator ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
