Re: [DISCUSS] Standardize Java style
My two cents: it’s much, much more important to have a standard style (enforced automatically) than what that style is. People should come into this expecting to compromise their personal preferences.
> On Aug 28, 2018, at 10:29 AM, Bryan Cutler <cutlerb@xxxxxxxxx> wrote:
> Sounds good Li. I just wanted to make sure we don't get locked into some
> style choices just because google does it that way. It's a good starting
> point though, so lets see how it looks applying the rules as part of
> ARROW-1688 and I'll raise anything that seems like it needs some discussion.
> On Tue, Aug 28, 2018 at 6:27 AM Li Jin <ice.xelloss@xxxxxxxxx> wrote:
>> I agree that the some every google style rules might not be the best fit
>> for Arrow project (e.g. import order rules). Like you said, we can use
>> google style as a base line and tweak specific rules if necessary.
>> On Mon, Aug 27, 2018 at 9:15 PM Bryan Cutler <cutlerb@xxxxxxxxx> wrote:
>>> Thanks for bringing this discussion up Li. I think we can use an existing
>>> style guide as a starting point, but ultimately we as a community should
>>> decide how to best fit it for the project. I believe we already have the
>>> google checkstlye as our Java rules configuration file, but already off
>>> bat the import rules seem insufficient. For example, they now have a
>>> import group that would encompass all third-party and Arrow imports, and
>>> the section of the guide about imports in your link is empty. I know a
>>> of style is a matter of preference, so I would encourage anyone to follow
>>> and comment in the JIRA at
>>> where I will be making style fixes incrementally in sub-tasks. Whatever
>>> style we choose does not have to be set in stone and can always be
>>> in the future, however I don't think we should point people to a
>>> third-party guide as official reference.
>>> On Mon, Aug 27, 2018 at 8:09 AM Li Jin <ice.xelloss@xxxxxxxxx> wrote:
>>>> Hi All,
>>>> Bryan Cutler has started a PR to fix Java checkstyle warning (Thank you
>>>> Bryan!). In my experience style is something hard to get consensus on
>>>> to personal preference, so I wonder if we can pick a well known style
>>>> (say google style: https://google.github.io/styleguide/javaguide.html)
>>>> minimize the discussion on styles?
>>>> What do other people think?