[
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)