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

Adam Szita commented on HIVE-18084:
-----------------------------------

Right now I think we shouldn't make warnings fail the checkstyle build. I 
intend to move forward with the Yetus<>ptest integration this week, and I think 
it will be enough of a change already that people would start seeing their 
checkstyle violations as warnings in the jira comments.

In the - not immediate - future we might want to make this more strict and 
actually fail the patch itself for certain style warnings, but right now I 
think we're not there yet :)

> Upgrade checkstyle version to support lambdas
> ---------------------------------------------
>
>                 Key: HIVE-18084
>                 URL: https://issues.apache.org/jira/browse/HIVE-18084
>             Project: Hive
>          Issue Type: Sub-task
>            Reporter: Adam Szita
>            Assignee: Adam Szita
>             Fix For: 3.0.0
>
>         Attachments: HIVE-18084.0.patch, diff.txt
>
>
> Current version does not support lambdas in source files so it skips them. We 
> need to upgrade checkstyle version to fix this.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to