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

ASF GitHub Bot commented on CALCITE-2352:
-----------------------------------------

Github user laurentgo commented on the issue:

    https://github.com/apache/calcite-avatica/pull/56
  
    From a public API perspective, there should not be: checkstyle only 
complains if constructor has a greater visibility than the class itself. The 
only exception has been the factory which is accessed through reflection (and 
the code assumes the constructor is public)


> Upgrade checkstyle to support Java8 lambdas
> -------------------------------------------
>
>                 Key: CALCITE-2352
>                 URL: https://issues.apache.org/jira/browse/CALCITE-2352
>             Project: Calcite
>          Issue Type: Bug
>          Components: avatica
>            Reporter: Laurent Goujon
>            Priority: Major
>
> Version of checkstyle used by Avatica (5.7) doesn't support constructs from 
> Java8 like lambdas and static methods in interfaces.
> This would result in lots of small changes to adjust to the stricter rules.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to