Mailing-List: contact dev-help@flink.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flink.apache.org
MIME-Version: 1.0
In-Reply-To: 
 <CAGr9p8B7s_B6yyqzN+4x5Q8NDeDsLVjW7ZDFpQTW6pcFL7D-7g@mail.gmail.com>
References: 
 <CAGr9p8B7s_B6yyqzN+4x5Q8NDeDsLVjW7ZDFpQTW6pcFL7D-7g@mail.gmail.com>
Date: Tue, 28 Jul 2015 20:19:27 +0200
Message-ID: 
 <CAGWx-_uF2VqTqS7ca+8wKXZBTmc4dbcG4F6UtVQbU=fQ+Z-phw@mail.gmail.com>
Subject: Re: Revert 78fd2146dd until we have consensus for FLINK-2419?
From: Kostas Tzoumas <ktzoumas@apache.org>
To: "dev@flink.apache.org" <dev@flink.apache.org>
Content-Type: multipart/alternative; boundary=001a11466196b31cfa051bf38200

--001a11466196b31cfa051bf38200
Content-Type: text/plain; charset=UTF-8

I'm probably lacking a bit of context, but by reading your conversation at
JIRA it seems to me that commit

https://github.com/apache/flink/commit/78fd2146dd00da1130910d9f23f09e2504854ef7

does not contain a test, and Robert is asking for a test which means that
we do not have consensus. If this was a pull request and not a commit it
would not be merged until consensus was reached, so my opinion is that the
same should happen now: the commit should be reverted, test(s) should be
added, and then merged again (or better submitted as a PR). More so as this
is deep in the core system code.

Kostas


On Tue, Jul 28, 2015 at 8:01 PM, Robert Metzger <rmetzger@apache.org> wrote:

> Hi,
>
> I'm a bit unhappy how we were handling
> https://issues.apache.org/jira/browse/FLINK-2419 today.
>
> I raised a concern in the JIRA because the commit for the fix didn't
> contain any tests. Our coding guidelines [1] imply that every feature
> should have tests. Apparently there were not enough tests for the two bugs
> fixed with commit 78fd2146dd.
>
> Also, Gyula's answer sounds like he is not willing to add tests right now.
>
> I can not remember if we ever reverted a commit in the Flink community, but
> in my understanding, this is how ASF projects are doing lazy consensus for
> commits-without-PR.
> So if there is a disagreement in the associated JIRA, we revert the fix
> until there is an agreement.
>
> In this case, I did not immediately revert the commit, because I would like
> to see whether others in the community agree with me.
>
>
> What do you think how we should handle cases like this one in the future?
>
> I think its very important for committers and PMC members to be a good
> example when it comes to following our own rules. Otherwise, how can we ask
> our contributors to adhere to these rules?
>
>
> My suggestion to resolve this situation is the following:
> - Revert commit 78fd2146dd
> - open pull requests for FLINK-2419 and FLINK-2423 (with tests of course),
> review and merge them.
>
>
>
> Best,
> Robert
>
> [1] http://flink.apache.org/coding-guidelines.html
>

--001a11466196b31cfa051bf38200--