Return-Path: X-Original-To: apmail-flink-dev-archive@www.apache.org Delivered-To: apmail-flink-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 86408105E5 for ; Mon, 16 Mar 2015 16:37:06 +0000 (UTC) Received: (qmail 54569 invoked by uid 500); 16 Mar 2015 16:37:06 -0000 Delivered-To: apmail-flink-dev-archive@flink.apache.org Received: (qmail 54504 invoked by uid 500); 16 Mar 2015 16:37:06 -0000 Mailing-List: contact dev-help@flink.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@flink.apache.org Delivered-To: mailing list dev@flink.apache.org Received: (qmail 54492 invoked by uid 99); 16 Mar 2015 16:37:06 -0000 Received: from mail-relay.apache.org (HELO mail-relay.apache.org) (140.211.11.15) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 16 Mar 2015 16:37:06 +0000 Received: from mail-qc0-f180.google.com (mail-qc0-f180.google.com [209.85.216.180]) by mail-relay.apache.org (ASF Mail Server at mail-relay.apache.org) with ESMTPSA id 17EF41A02E4 for ; Mon, 16 Mar 2015 16:37:06 +0000 (UTC) Received: by qcyi15 with SMTP id i15so48881597qcy.0 for ; Mon, 16 Mar 2015 09:37:05 -0700 (PDT) MIME-Version: 1.0 X-Received: by 10.140.150.149 with SMTP id 143mr79986286qhw.4.1426523825097; Mon, 16 Mar 2015 09:37:05 -0700 (PDT) Received: by 10.229.123.66 with HTTP; Mon, 16 Mar 2015 09:37:05 -0700 (PDT) In-Reply-To: References: Date: Mon, 16 Mar 2015 17:37:05 +0100 Message-ID: Subject: Re: [DISCUSS] Issues with heterogeneity of the code From: Till Rohrmann To: dev@flink.apache.org Content-Type: multipart/alternative; boundary=001a1135510cdbd52205116a7501 --001a1135510cdbd52205116a7501 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Do we already enforce the official Scala style guide strictly? On Mon, Mar 16, 2015 at 4:57 PM, Aljoscha Krettek wrote: > I'm already always sticking to the official Scala style guide, with the > exception of 100 line length. > On Mar 16, 2015 3:27 PM, "Till Rohrmann" wrote: > > > +1 for stricter Java code styles. I haven't looked into the Google Code > > Style but maybe we make it easier for new contributors if we apply a > coding > > style which is somehow known. > > > > +1 for line length of 100 for Scala code. I think it makes code review = on > > GitHub easier. > > > > For the Scala style, we could stick to official style guidelines [1]. > > > > [1] http://docs.scala-lang.org/style/ > > > > On Mon, Mar 16, 2015 at 3:06 PM, Hermann G=C3=A1bor > > wrote: > > > > > +1 for the stricter Java code styles. > > > > > > We should not forget about providing code formatter settings for > Eclipse > > > and Intellij IDEA (as mentioned above). > > > That would help a lot. > > > > > > (Of course if we'll use Google Code Style, they already provide such > > files > > > < > > > > > > https://code.google.com/p/google-styleguide/source/browse/trunk/intellij-= java-google-style.xml > > > > > > > .) > > > > > > On Mon, Mar 16, 2015 at 2:45 PM Alexander Alexandrov < > > > alexander.s.alexandrov@gmail.com> wrote: > > > > > > > +1 for not limiting the line length. > > > > > > > > 2015-03-16 14:39 GMT+01:00 Stephan Ewen : > > > > > > > > > +1 for not limiting the line length. Everyone should have a good > > sense > > > to > > > > > break lines. When in exceptional cases people violate this, it is > > > usually > > > > > for a good reason. > > > > > > > > > > On Mon, Mar 16, 2015 at 2:18 PM, Maximilian Michels < > mxm@apache.org> > > > > > wrote: > > > > > > > > > > > +1 for enforcing a more strict Java code style. However, let's > not > > > > > > introduce a line legth of 100 like in Scala. I think that's > hurting > > > > > > readability of the code. > > > > > > > > > > > > On Sat, Mar 14, 2015 at 4:41 PM, Ufuk Celebi > > wrote: > > > > > > > > > > > > > On Saturday, March 14, 2015, Aljoscha Krettek < > > aljoscha@apache.org > > > > > > > > > > wrote: > > > > > > > > > > > > > > > I'm in favor of strict coding styles. And I like the google > > > style. > > > > > > > > > > > > > > > > > > > > > +1 I would like that. We essentially all agree that we want > more > > > > > > > homogeneity and I think strict rules are the only way to go. > > Since > > > > this > > > > > > is > > > > > > > a very subjective matter it makes sense to go with something > > > > (somewhat) > > > > > > > well > > > > > > > established like the Google code style. > > > > > > > > > > > > > > > > > > > > > > > > > > > > --001a1135510cdbd52205116a7501--