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

Dong Li commented on KYLIN-2745:
--------------------------------

Hi Shaofeng and Nirav,

In fact, these setting files are used for code formatting.

In this document, eclipse formatter is suggested for non-eclipse users:
http://kylin.apache.org/development/dev_env.html

So I would suggest to keep these files, and also remove them from .gitignore.

Thanks.

> Remove eclipse .settings directory from each module
> ---------------------------------------------------
>
>                 Key: KYLIN-2745
>                 URL: https://issues.apache.org/jira/browse/KYLIN-2745
>             Project: Kylin
>          Issue Type: Task
>    Affects Versions: Future, v2.0.0
>            Reporter: nirav patel
>            Assignee: nirav patel
>             Fix For: Future
>
>         Attachments: KYLIN-2745.patch
>
>
> I think this directory should not be part of git repo at all. It should be 
> listed in .gitignore file. 



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

Reply via email to