Return-Path: X-Original-To: apmail-pig-dev-archive@www.apache.org Delivered-To: apmail-pig-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 DEAB4960D for ; Wed, 28 Mar 2012 05:43:52 +0000 (UTC) Received: (qmail 12939 invoked by uid 500); 28 Mar 2012 05:43:52 -0000 Delivered-To: apmail-pig-dev-archive@pig.apache.org Received: (qmail 12538 invoked by uid 500); 28 Mar 2012 05:43:47 -0000 Mailing-List: contact dev-help@pig.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@pig.apache.org Delivered-To: mailing list dev@pig.apache.org Received: (qmail 12493 invoked by uid 99); 28 Mar 2012 05:43:45 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 28 Mar 2012 05:43:45 +0000 X-ASF-Spam-Status: No, hits=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of dvryaboy@gmail.com designates 209.85.216.46 as permitted sender) Received: from [209.85.216.46] (HELO mail-qa0-f46.google.com) (209.85.216.46) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 28 Mar 2012 05:43:39 +0000 Received: by qaeb19 with SMTP id b19so4203830qae.5 for ; Tue, 27 Mar 2012 22:43:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type:content-transfer-encoding; bh=VTu/6E7RR558IQ9xUfdlNhGTviIuin4v7mE4NiPOnfI=; b=gJBMm09iHkdKVyyGFpjtKIh6CXxMGcvxWiYQktoMJ659fLgkxAVreesJhdVc15kz8W a6j9IDYZslBTbsgzpFUeIvwdzec6MB/li5nWBW+kKXyQMrAUPJ3IhXJLg28fX6CozxbH NoXw9Rcz9bh1LP7nlQZbq+CGX2sxXSo9FJUBNB7Row4H5RFetrKLeuQhhse9dz0xdb2P 1eom8Rdg68vk3IheTQrPbpE7O+hZ7sE8OURudmt/0PAm5jLGZwIrokp0Q3tnWuC8+elU LrwuZOgzq6OLvYFMyliGskzThvUNjm6zIBAAyKYqkFUrovcQT7xK5WELrRCmXcGwH5+s cC1w== MIME-Version: 1.0 Received: by 10.224.180.201 with SMTP id bv9mr36305306qab.69.1332913397902; Tue, 27 Mar 2012 22:43:17 -0700 (PDT) Received: by 10.229.98.204 with HTTP; Tue, 27 Mar 2012 22:43:17 -0700 (PDT) In-Reply-To: References: Date: Tue, 27 Mar 2012 22:43:17 -0700 Message-ID: Subject: Re: Making git the repo of choice for Pig? From: Dmitriy Ryaboy To: dev@pig.apache.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Could you enumerate the problems you have with text patches? I can think of 1, but would like to hear your list. Dmitriy On Tue, Mar 27, 2012 at 2:48 PM, Russell Jurney wrote: > Using text patches is the problem as I see it. =A0We should be using... > something else. =A0Doesn't git's functionality enable phasing out text > patches? > > On Tue, Mar 27, 2012 at 11:16 AM, Scott Carey wr= ote: > >> On the Avro project, several folk generate a patch using git and upload >> that to the JIRA, it patches fine using unix 'patch'. >> Why do you need to switch to svn to generate the patch? >> >> A committer needs to apply the patch to a location checked out with svn, >> but that is trivial. >> >> On 3/27/12 2:26 AM, "Gianmarco De Francisci Morales" >> wrote: >> >> >My 2 cents, based on the S4 project experience so far (they have git) i= s >> >+1 >> >but later. >> > >> >The process with git is still a bit fuzzy and not fully integrated and >> >standardized like the one with svn. >> >I personally use git for all my dev work with Pig, but then switch to s= vn >> >to generate the final patch. A bit cumbersome but not so bad. >> > >> >Switching now will generate confusion and is not that urgent in my humb= le >> >opinion. >> > >> >Cheers, >> >-- >> >Gianmarco >> > >> > >> > >> >On Sat, Mar 24, 2012 at 01:27, Dmitriy Ryaboy wrot= e: >> > >> >> There is a check box you check when you upload a patch. If we committ= ed >> >> without verifying you checked it, thats an unfortunate oversight. >> >> >> >> No, you cant send a pull request via github to apache. That's a githu= b >> >> (commercial entity) feature, not a git (open source vcs) feature. >> >> >> >> Yes you can post a link on a Jira and ask people for review prior to >> >> submitting a formal patch. =A0No that's not review board or Jira >> >>integration. >> >> >> >> >> >> >> >> On Mar 22, 2012, at 5:59 PM, Russell Jurney >> >> wrote: >> >> >> >> > That is fine. =A0Only committers can merge pull requests. =A0Not a >> >>problem. >> >> > >> >> > I've never signed anything, and committers take my patches. =A0Work= s the >> >> same. >> >> > >> >> > On Thu, Mar 22, 2012 at 3:33 PM, Scott Carey > >> >wrote: >> >> > >> >> >> >> >> >> There is a legal issue as well. Uploading a patch to JIRA requires >> >>the >> >> >> user to grant apache license to use the code. =A0A pull request fr= om >> >> github >> >> >> does not. >> >> >> This is all being hashed out elsewhere, none of us here are expert= s. >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> On 3/22/12 2:08 PM, "Russell Jurney" >> >>wrote: >> >> >> >> >> >>> If there is already a trial going, the project would benefit from >> >> >>> switching >> >> >>> to git, as we could then (I think?) accept pull requests on githu= b. >> >> =A0Maybe >> >> >>> not, I'm not familiar with integrating multiple gits. =A0Barrier = of >> >>entry >> >> >>> for >> >> >>> contributing would then drop dramatically, making happy for all. >> >> >>> >> >> >>> On Thu, Mar 22, 2012 at 1:58 PM, Daniel Dai >> >> >> wrote: >> >> >>> >> >> >>>> This can be done in the future but I don't see there is an urgen= t >> >>need >> >> >>>> for that. I'd rather wait for Apache to standardize the process >> >>and we >> >> >>>> can easily follow and switch, rather than become the first sever= al >> >>to >> >> >>>> try and solve the infrastructure problem. >> >> >>>> >> >> >>>> Daniel >> >> >>>> >> >> >>>> On Thu, Mar 22, 2012 at 12:02 PM, Jonathan Coveney < >> >> jcoveney@gmail.com> >> >> >>>> wrote: >> >> >>>>> If people don't know git or svn, they are going to have to lear= n >> >>one >> >> >>>> or >> >> >>>> the >> >> >>>>> other if they want to work with open source projects. It might = as >> >> >>>> well be >> >> >>>>> git, since that's where things are going. >> >> >>>>> >> >> >>>>> 2012/3/22 Scott Carey >> >> >>>>> >> >> >>>>>> Apache is currently doing a trial with git as the primary repo >> >>using >> >> >>>> the >> >> >>>>>> CouchDB project. =A0Other projects may be able to volunteer to= join >> >> the >> >> >>>>>> trial, I'm not sure. >> >> >>>>>> >> >> >>>>>> see http://git-wip-us.apache.org/ >> >> >>>>>> >> >> >>>>>> I suggest that if you are interested in this join the apache >> >> >>>>>> infrastructure-dev list. >> >> >>>>>> >> >> >>>>>> >> >> >>>>>> >> >> >>>>>> On 3/21/12 9:57 PM, "Russell Jurney" >> >> >>>> wrote: >> >> >>>>>> >> >> >>>>>>> What's the work? >> >> >>>>>>> >> >> >>>>>>> Russell Jurney http://datasyndrome.com >> >> >>>>>>> >> >> >>>>>>> On Mar 21, 2012, at 9:39 PM, Dmitriy Ryaboy > > >> >> >>>> wrote: >> >> >>>>>>> >> >> >>>>>>>> All that's left is for someone to volunteer to do the work. >> >> >>>>>>>> >> >> >>>>>>>> On Mar 21, 2012, at 9:17 PM, Russell Jurney < >> >> >>>> russell.jurney@gmail.com> >> >> >>>>>>>> wrote: >> >> >>>>>>>> >> >> >>>>>>>>> What do we have to do to be the first real project that use= s >> >>git? >> >> >>>>>>>>> Let's do that. Or, let's just sink svn to github. >> >> >>>>>>>>> >> >> >>>>>>>>> It will propel the project forward. >> >> >>>>>>>>> >> >> >>>>>>>>> Russell Jurney >> >> >>>>>>>>> twitter.com/rjurney >> >> >>>>>>>>> russell.jurney@gmail.com >> >> >>>>>>>>> datasyndrome.com >> >> >>>>>>>>> >> >> >>>>>>>>> On Mar 21, 2012, at 9:04 PM, Bill Graham >> >> >> >> >>>> wrote: >> >> >>>>>>>>> >> >> >>>>>>>>>> I think this is the tweet (lacking the entire discussion t= ree >> >> >>>> that >> >> >>>> we >> >> >>>>>>>>>> had >> >> >>>>>>>>>> re git) where I was pointing out that some incubator proje= cts >> >> >>>> were >> >> >>>>>>>>>> using >> >> >>>>>>>>>> git. >> >> >>>>>>>>>> >> >> >>>>>>>>>> https://twitter.com/#!/billgraham/status/17474419940773888= 0 >> >> >>>>>>>>>> >> >> >>>>>>>>>> >> >> >>>>>>>>>> On Wed, Mar 21, 2012 at 12:47 PM, Russell Jurney >> >> >>>>>>>>>> wrote: >> >> >>>>>>>>>> >> >> >>>>>>>>>>> I think this changed, and you are now allowed. =A0Looking= for >> >>the >> >> >>>>>>>>>>> tweets. >> >> >>>>>>>>>>> >> >> >>>>>>>>>>> Russell Jurney http://datasyndrome.com >> >> >>>>>>>>>>> >> >> >>>>>>>>>>> On Mar 21, 2012, at 9:36 AM, Alan Gates >> >>> >> >>> >> >> >>>>>>>>>>> wrote: >> >> >>>>>>>>>>> >> >> >>>>>>>>>>>> AFAIK Apache still does not support git as a primary >> >> >>>> repository. >> >> >>>>>>>>>>>> You >> >> >>>>>>>>>>> can use the git mirror, which Pig does. =A0If this has ch= anged >> >> >>>> (or >> >> >>>>>>>>>>> when it >> >> >>>>>>>>>>> does), I'm +0 on changing, by which I mean I don't care >> >>which >> >> >>>> we >> >> >>>> use. >> >> >>>>>>>>>>>> >> >> >>>>>>>>>>>> Alan. >> >> >>>>>>>>>>>> >> >> >>>>>>>>>>>> On Mar 20, 2012, at 11:22 PM, Jonathan Coveney wrote: >> >> >>>>>>>>>>>> >> >> >>>>>>>>>>>>> Would anyone be opposed to doing this? I think you can >> >>apply >> >> >>>> to >> >> >>>> do >> >> >>>>>>>>>>>>> this >> >> >>>>>>>>>>> via >> >> >>>>>>>>>>>>> Apache, no? It would be nice to make git the first clas= s >> >> >>>> citizen of >> >> >>>>>>>>>>>>> choice... forcing developers to use svn to work with pi= g >> >>is >> >> >>>> just >> >> >>>>>>>>>>>>> another >> >> >>>>>>>>>>>>> barrier, imho. I thought this had been bandied about on >> >> >>>> Twitter >> >> >>>>>>>>>>>>> and the >> >> >>>>>>>>>>>>> like, but not sure if it's been discussed seriously on = the >> >> >>>>>>>>>>>>> listserv. >> >> >>>>>>>>>>>>> >> >> >>>>>>>>>>>>> Would love y'all's opinions on the matter >> >> >>>>>>>>>>>>> Jon >> >> >>>>>>>>>>>> >> >> >>>>>>>>>>> >> >> >>>>>>>>>> >> >> >>>>>>>>>> >> >> >>>>>>>>>> >> >> >>>>>>>>>> -- >> >> >>>>>>>>>> *Note that I'm no longer using my Yahoo! email address. >> >>Please >> >> >>>> email >> >> >>>>>>>>>> me at >> >> >>>>>>>>>> billgraham@gmail.com going forward.* >> >> >>>>>> >> >> >>>>>> >> >> >>>> >> >> >>> >> >> >>> >> >> >>> >> >> >>> -- >> >> >>> Russell Jurney twitter.com/rjurney russell.jurney@gmail.com >> >> >>> datasyndrome.com >> >> >> >> >> >> >> >> > >> >> > >> >> > -- >> >> > Russell Jurney twitter.com/rjurney russell.jurney@gmail.com >> >> datasyndrome.com >> >> >> >> > > > -- > Russell Jurney twitter.com/rjurney russell.jurney@gmail.com datasyndrome.= com