You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by "Vinoth Chandar (Jira)" <ji...@apache.org> on 2021/01/09 20:06:00 UTC

[jira] [Commented] (HUDI-304) Bring back spotless plugin

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

Vinoth Chandar commented on HUDI-304:
-------------------------------------

[~xushiyan] FYI. Flink recently reformatted everything, with code style, spotless, IDE being in sync. might be worth understanding what they did and if we can directly reuse

cc [~xleesf] [~vinoyang] 

> Bring back spotless plugin 
> ---------------------------
>
>                 Key: HUDI-304
>                 URL: https://issues.apache.org/jira/browse/HUDI-304
>             Project: Apache Hudi
>          Issue Type: Task
>          Components: Code Cleanup, Testing
>            Reporter: Balaji Varadarajan
>            Assignee: Raymond Xu
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 0.7.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> spotless plugin has been turned off as the eclipse style format it was referencing was removed due to compliance reasons. 
> We use google style eclipse format with some changes
> 90c90
> < <setting id="org.eclipse.jdt.core.formatter.lineSplit" value="100"/>
> ---
> > <setting id="org.eclipse.jdt.core.formatter.lineSplit" value="120"/>
> 242c242
> < <setting id="org.eclipse.jdt.core.formatter.comment.line_length" value="100"/>
> ---
> > <setting id="org.eclipse.jdt.core.formatter.comment.line_length" value="120"/>
>  
> The eclipse style sheet was originally obtained from [https://github.com/google/styleguide] which CC -By 3.0 license which is not compatible for source distribution (See [https://www.apache.org/legal/resolved.html#cc-by]) 
>  
> We need to figure out a way to bring this back
>  
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)