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

Julian Hyde commented on CALCITE-3559:
--------------------------------------

Please don't commit this until we get to consensus. This is a step backwards. 
HydromaticFileSetCheck is not perfect, but it checks quite a few things that 
checkstyle cannot. I wrote it because checkstyle is not easily extensible 
either.

> Drop HydromaticFileSetCheck from Checkstyle configuration
> ---------------------------------------------------------
>
>                 Key: CALCITE-3559
>                 URL: https://issues.apache.org/jira/browse/CALCITE-3559
>             Project: Calcite
>          Issue Type: Task
>          Components: build
>    Affects Versions: 1.21.0
>         Environment: HydromaticFileSetCheck is not customizable, and it is 
> not supported.
> HydromaticFileSetCheck is not compatible with Checkstyle 8.x which resolves 
> concurrency issues.
> Lots of checks implemented in HydromaticFileSetCheck can be automated (==the 
> software could automatically fix the code rather than complain).
> So we should drop HydromaticFileSetCheck
>            Reporter: Vladimir Sitnikov
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 20m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to