You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kylin.apache.org by "Dong Li (JIRA)" <ji...@apache.org> on 2016/11/23 00:30:58 UTC

[jira] [Commented] (KYLIN-2194) Fix checkstyle warning in DerivedFilterTranslator.java

    [ https://issues.apache.org/jira/browse/KYLIN-2194?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15688419#comment-15688419 ] 

Dong Li commented on KYLIN-2194:
--------------------------------

already fixed in commit dd9c2c9f504e462f71ed93cf317ef4a46bf6237f

> Fix checkstyle warning in DerivedFilterTranslator.java
> ------------------------------------------------------
>
>                 Key: KYLIN-2194
>                 URL: https://issues.apache.org/jira/browse/KYLIN-2194
>             Project: Kylin
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: Dong Li
>            Priority: Trivial
>
> {code}
> [INFO] Starting audit...
> /mnt/disk2/a/kylin/core-storage/src/main/java/org/apache/kylin/storage/translate/DerivedFilterTranslator.java:89:121: error: ',' is not followed by whitespace.
> [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:2.17:check (validate) on project kylin-core-storage: Failed during checkstyle execution: There is 1 error reported by Checkstyle 6.11.2 with dev-support/checkstyle.xml ruleset. -> [Help 1]
> {code}
> There should be whitespace between comma and "//"



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)