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

Adam Szita commented on HIVE-17995:
-----------------------------------

[~alangates], I agree, it makes sense to copy the checkstyle config folder 
anyway since at some point when we separate HMS out from under Hive it will 
need it anyway.
Attached new patch with this change: [^HIVE-17995.1.patch]

> Run checkstyle on standalone-metastore module with proper configuration
> -----------------------------------------------------------------------
>
>                 Key: HIVE-17995
>                 URL: https://issues.apache.org/jira/browse/HIVE-17995
>             Project: Hive
>          Issue Type: Sub-task
>            Reporter: Adam Szita
>            Assignee: Adam Szita
>         Attachments: HIVE-17995.0.patch, HIVE-17995.1.patch
>
>
> Maven module standalone-metastore is obviously not connected to Hive root 
> pom, therefore if someone (or an automated Yetus check) runs {{mvn 
> checkstyle}} it will not consider Hive-specific checkstyle settings (e.g. 
> validates row lengths against 80, not 100)
> We need to make sure standalone-metastore pom has the proper checkstyle 
> configuration



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

Reply via email to