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

Kevin Risden commented on CALCITE-2352:
---------------------------------------

[~laurentgo] - Looks like this commit is causing build issues now?

 

[https://builds.apache.org/job/Calcite-Avatica-Master/71/]

 

I was able to reproduce this locally as well:

mvn verify javadoc:javadoc

 

[ERROR] 
target/filtered-java-src/org/apache/calcite/avatica/util/FilteredConstants.java:[0]
 (javadoc) JavadocPackage: Missing package-info.java file.
[ERROR] 
target/filtered-java-src/org/apache/calcite/avatica/util/FilteredConstants.java:[22,1]
 (design) HideUtilityClassConstructor: Utility classes should not have a public 
or default constructor.

 

[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-checkstyle-plugin:3.0.0:check (validate) on 
project avatica-core: You have 2 Checkstyle violations. -> [Help 1]

 

Did the same commit from the PR get merged or something else?

> 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
>            Assignee: Laurent Goujon
>            Priority: Major
>             Fix For: avatica-1.12.0
>
>
> 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