Mailing-List: contact dev-help@zookeeper.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@zookeeper.apache.org
MIME-Version: 1.0
In-Reply-To: <CA+i0x1+=rKGqcp4jTqm+tNG2K+2steMefPj-6n-LGNyQz_3CJA@mail.gmail.com>
References: <CAC==nPTNhTmo7ymBcr43RvX0WKVzWaDxdrNBOvYxj-uNKNSLvQ@mail.gmail.com>
 <CAC==nPRZnE=JpssO4gXdKMDdcK2KuD1HG-o40MegwAOAF=OiuA@mail.gmail.com>
 <CAC==nPQ3BWwbVU4fi2GoP5FZ=VrDoa1MX5GOqiOq736bz1JssA@mail.gmail.com>
 <CAC==nPSwu6nuHhYRAy3+QaURAoJpidwHNitJSXgXgMUBTUzBCQ@mail.gmail.com>
 <CAC==nPTh397qyvKe4UXhC8608H=E76sq8P0eo=OSADmQb7gqnw@mail.gmail.com>
 <BF388FE1-C6EF-4045-88AC-49171658B365@apache.org> <CAC==nPSmaCvGw=dLQRsZYfAW5BVHSZQUie-TN7dt+JBfv48cYw@mail.gmail.com>
 <5D89E1B9-F1C3-48AB-AEB7-33623C425108@apache.org> <CA+i0x1Lj7EKfkHn04vm5wAHu+ETwXnWv+ieR+Kf3N6EWkrapDg@mail.gmail.com>
 <CAKR25ERxWNAMDta4UW1cV3N=9tF2aGvErJW-X=1XHfMK=shdWA@mail.gmail.com>
 <CA+i0x1J4=Hht-sQw5zOugm2Li_Owt8Lx7F5j1NoVOb3btNQrdA@mail.gmail.com>
 <412F8FEF-0141-4EE2-909F-EF5BD5BBA61C@apache.org> <B059DAF8-C13B-4E10-B1EB-512A1E0BEE59@apache.org>
 <BEA6BF03-8FC4-42EC-BBAC-9480483BDDBC@apache.org> <CAC==nPQg1Ai9bbh8GJmn7ryoyMW7TcF++zmv_HWG4MjOULeWAw@mail.gmail.com>
 <953D1750-48BB-4044-BD87-D13C58799CFC@apache.org> <CAC==nPSdPvhDO2zrsOxwO9YE2hUkH+yvY3kUEx7mu7fai3J0xw@mail.gmail.com>
 <A2D7BBEC-E9F7-4476-90FF-EB536D29F957@apache.org> <C6688AC7-F525-477C-84FD-5213BC665CF8@apache.org>
 <CAMUR=WgZE6Lm5a_DhFLPZ_KwW9kEAVU8jtfrZP2MD-CQ_TUVnA@mail.gmail.com>
 <CAC==nPTSC530Rm8sOS7zeeoq21CPd4_7nmRGyn-TC9WDScxvkA@mail.gmail.com> <CA+i0x1+=rKGqcp4jTqm+tNG2K+2steMefPj-6n-LGNyQz_3CJA@mail.gmail.com>
From: Flavio P JUNQUEIRA <fpj@apache.org>
Date: Tue, 8 Nov 2016 18:37:36 -0800
Message-ID: <CAB5oV29CjA6Huz-XM0D_RnP=557d_cO_CDWAufHLF_DWBs31-A@mail.gmail.com>
Subject: Re: [DISCUSS] QA github pre-commit queue
To: Zookeeper <dev@zookeeper.apache.org>
Content-Type: multipart/alternative; boundary=001a114fd420d339440540d5235f
archived-at: Wed, 09 Nov 2016 02:37:39 -0000

--001a114fd420d339440540d5235f
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

I don't have any strong argument for keeping the Jira patches other than
the fact that this is what we have been doing since the project was
created. If there is anyone who do not want to use github in the community,
please speak up.

-Flavio

On Mon, Nov 7, 2016 at 9:49 AM, Michael Han <hanm@cloudera.com> wrote:

> +1 for disabling jira qa and only support pull request for code change
> contributions. Besides making support easier this approach is also aligne=
d
> with what Spark and Kafka is doing, and being consistent across Apache
> projects regarding how to use PR seems a good thing to do.
>
> >> have the tool upload the *.patch file to Jira for archiving purposes.
> I think nothing will prevent a user submit a patch file to JIRA with our
> script changes, so the functionality of archiving patches will still work=
.
> Though, I noticed that Kafka [1] and Spark [2] explicitly stated that do
> not include patch file in JIRA for code contributions, so probably we'd d=
o
> this too for consistency purpose? Are there any benefit of archiving
> patches given we prefer (or actually require) pull request instead of
> patches?
>
> [1]
> https://cwiki.apache.org/confluence/display/KAFKA/
> Contributing+Code+Changes#ContributingCodeChanges-PullRequest
> [2]
> https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark#
> ContributingtoSpark-PullRequest
>
> On Mon, Nov 7, 2016 at 8:04 AM, Edward Ribeiro <edward.ribeiro@gmail.com>
> wrote:
>
> > I am +1 about having patches submitted via PRs. IMHO, we should disable
> the
> > Jira QA altogether, but have the tool upload the *.patch file to Jira f=
or
> > archiving purposes.
> >
> > On Sun, Nov 6, 2016 at 6:42 PM, Ra=C3=BAl Guti=C3=A9rrez Segal=C3=A9s <
> > rgs@itevenworks.net>
> > wrote:
> >
> > > On 6 November 2016 at 11:54, Flavio Junqueira <fpj@apache.org> wrote:
> > >
> > > > ZOOKEEPER-2624 has been merged, thank Raul, Ben and Michael for
> > > reviewing.
> > > >
> > > > The QA for pull requests should be working for pull requests agains
> > > > master, but let's keep an eye and polish any rough edges that might
> > still
> > > > be there.
> > > >
> > > > With ZOOKEEPER-2624 in, there is one last major decision we need to
> > make
> > > > to wrap this up. The pull request QA currently do not make a jira
> patch
> > > > available. This is intentional because making it patch available wi=
ll
> > > > trigger the original Jira QA, which will be confusing because we wi=
ll
> > > see a
> > > > failure (I haven't tested, but I think that's what's going to
> happen).
> > If
> > > > we change the script to make the Jira patch available, then we need
> to
> > > > either:
> > > >
> > > > 1- Disable the Jira QA altogether, which means that we will only ha=
ve
> > > pull
> > > > request QA available
> > > > 2- Make the Jira QA script spot that there is a pull request
> available
> > > and
> > > > not build it.
> > > >
> > > > I'm wondering if folks would be ok with only having patches submitt=
ed
> > via
> > > > pull requests or if we should continue to support the old Jira QA.
> > > >
> > >
> > > I am +1 on only having patches submitted via PRs, it's simpler to onl=
y
> > have
> > > to support one method. Thanks Flavio for making this happen!
> > >
> > >
> > > -rgs
> > >
> >
>
>
>
> --
> Cheers
> Michael.
>

--001a114fd420d339440540d5235f--