Return-Path: X-Original-To: apmail-accumulo-dev-archive@www.apache.org Delivered-To: apmail-accumulo-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 B4D9DFB2B for ; Fri, 14 Nov 2014 18:59:16 +0000 (UTC) Received: (qmail 56729 invoked by uid 500); 14 Nov 2014 18:59:16 -0000 Delivered-To: apmail-accumulo-dev-archive@accumulo.apache.org Received: (qmail 56689 invoked by uid 500); 14 Nov 2014 18:59:16 -0000 Mailing-List: contact dev-help@accumulo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@accumulo.apache.org Delivered-To: mailing list dev@accumulo.apache.org Received: (qmail 56674 invoked by uid 99); 14 Nov 2014 18:59:16 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 14 Nov 2014 18:59:16 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of mdrob@cloudera.com designates 209.85.218.44 as permitted sender) Received: from [209.85.218.44] (HELO mail-oi0-f44.google.com) (209.85.218.44) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 14 Nov 2014 18:58:48 +0000 Received: by mail-oi0-f44.google.com with SMTP id h136so12504562oig.31 for ; Fri, 14 Nov 2014 10:58:02 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:content-type; bh=5oQSoHHVsjbBuIbOYrmvj6DJJR2xWZfvPKYzbuRTfkQ=; b=OeFG/aUCFsmetbcxZj7NIqoGT+yKDEfxkESi5VfDwO1I++6H7ialKXBNzCDdvneTST VvxoDL7NuXAQ1XJ+rU6zmZWZ7LCFszX7cLTe5oG/ZXnx5Akhnha/yf1/oqMTon6WXXl2 R1q1c/7n90RqbF1YF8RGUnRutEG1ni6Ya8G+cRpLgWZeIp3UqbNFczJJCzjykcD2TC30 b2R97IES9oPhU/lHZLEo/CLu7sM6GTN06Wy97qYULrA0hOOhFEPTEjqDNkaLPwlrsyty 81S1xrzzT0kZLr0Nw3OEfjwr9YMYgXrwqlUE6R+HW51tXiO/8gapKPemtglLNrj3TyTI XxAw== X-Gm-Message-State: ALoCoQl7MZ0t/Mw7fnsgvVp2/ZfU2BdqnSKr6znN/BTOwvtEwhF4o5I1n+bYkpWAFWTd9qDINgAv X-Received: by 10.60.54.74 with SMTP id h10mr9409172oep.9.1415991482614; Fri, 14 Nov 2014 10:58:02 -0800 (PST) MIME-Version: 1.0 Received: by 10.60.156.74 with HTTP; Fri, 14 Nov 2014 10:57:42 -0800 (PST) In-Reply-To: References: <54623B88.6030506@gmail.com> <5463A779.3050609@gmail.com> <5463ADC7.2010906@gmail.com> <5463B2C0.40308@gmail.com> <5463BAE6.9070702@gmail.com> <54664688.4070503@gmail.com> From: Mike Drob Date: Fri, 14 Nov 2014 12:57:42 -0600 Message-ID: Subject: Re: Contribute Examples/Exercises To: Accumulo Dev List Content-Type: multipart/alternative; boundary=089e013a1ce8539fe20507d63593 X-Virus-Checked: Checked by ClamAV on apache.org --089e013a1ce8539fe20507d63593 Content-Type: text/plain; charset=UTF-8 Having done this in the past, I have a hard time suggesting this approach. As a single module, the code probably won't compile for most of the history that we preserve, so it's not like we'd be able to efficiently bisect or take advantage of the history. On Fri, Nov 14, 2014 at 12:29 PM, Keith Turner wrote: > We can try using git fitler-branch to create the repo and preserve history. > > On Fri, Nov 14, 2014 at 1:14 PM, Josh Elser wrote: > > > Since there's an opinion to create an examples repo instead of keeping > > them in the base project, I'm -0 as long we CI set up so that they don't > go > > silently into the night as I previously state as a concern. > > > > Some general questions for actually doing this: do we schedule the move > of > > the classes out of the main project for 1.7.0? Will this other repo > follow > > the same development practices as the project (e.g. branch names). How > will > > we release these examples? > > > > Can someone step up to make sure all of the above are completed/addressed > > and file the necessary INFRA JIRA issues? > > > > > > David Medinets wrote: > > > >> +1 > >> On Nov 14, 2014 11:18 AM, "Keith Turner" wrote: > >> > >> On Wed, Nov 12, 2014 at 4:52 PM, Corey Nolet > wrote: > >>> > >>> Josh, > >>>> > >>>> My worry with a contrib module is that, historically, code which goes > >>>>> > >>>> moves to a contrib is just one step away from the grave. > >>>> > >>>> You do have a good point. My hope was that this could be the beginning > >>>> of > >>>> our changing history so that we could begin to encourage the community > >>>> to > >>>> contribute their own source directly and give them an outlet for doing > >>>> > >>> so. > >>> > >>>> I understand that's also the intent of hosting open source repos under > >>>> > >>> ASF > >>> > >>>> to begin with- so I'm partial to either outcome. > >>>> > >>>> I think there's precedence for keeping them in core (as Christopher > had > >>>>> > >>>> mentioned, next to examples/simple) which would benefit people > >>>> externally > >>>> (more "how do I do X" examples) and internally (keep devs honest about > >>>> > >>> how > >>> > >>>> our APIs are implemented). > >>>> > >>>> I would think that would just require keeping the repos up to date as > >>>> versions change so they wouldn't get out of date and possibly > releasing > >>>> them w/ our other releases. > >>>> > >>>> > >>>> Wherever they end up living, thank you Adam for the contributions! > >>>> > >>>> I'll 2nd that. > >>> > >>> For the following reasons, I think it might be nice to move existing > >>> examples out of core into their own git repo(s). > >>> > >>> * Examples would be based on released version of Accumulo > >>> * Examples could easily be built w/o building all of Accumulo > >>> * As Sean said, this would keep us honest > >>> * The examples poms would serve as examples more than they do when > >>> part of > >>> Accumulo build > >>> * Less likely to use non public APIs in examples > >>> > >>> > >>> > >>>> > >>>> On Wed, Nov 12, 2014 at 2:54 PM, Josh Elser > >>>> > >>> wrote: > >>> > >>>> My worry with a contrib module is that, historically, code which goes > >>>>> moves to a contrib is just one step away from the grave. I think > >>>>> > >>>> there's > >>> > >>>> precedence for keeping them in core (as Christopher had mentioned, > next > >>>>> > >>>> to > >>>> > >>>>> examples/simple) which would benefit people externally (more "how do > I > >>>>> > >>>> do > >>> > >>>> X" examples) and internally (keep devs honest about how our APIs are > >>>>> implemented). > >>>>> > >>>>> Bringing the examples into the core also encourages us to grow the > >>>>> community which has been stagnant with respect to new committers for > >>>>> > >>>> about > >>>> > >>>>> 9 months now. > >>>>> > >>>>> > >>>>> Corey Nolet wrote: > >>>>> > >>>>> +1 for adding the examples to contrib. > >>>>>> > >>>>>> I was, myself, reading over this email wondering how a set of 11 > >>>>>> > >>>>> separate > >>>> > >>>>> examples on the use of Accumulo would fit into the core codebase- > >>>>>> especially as more are contributed over tinme. I like the idea of > >>>>>> > >>>>> giving > >>> > >>>> community members an outlet for contributing examples that they've > >>>>>> > >>>>> built > >>> > >>>> so > >>>>>> that we can continue to foster that without having to fit them in > the > >>>>>> > >>>>> core > >>>> > >>>>> codebase. It just seems more maintainable. > >>>>>> > >>>>>> > >>>>>> On Wed, Nov 12, 2014 at 2:19 PM, Josh Elser > >>>>>> > >>>>> wrote: > >>>> > >>>>> I'll take that as you disagree with my consideration of > >>>>>> > >>>>> "substantial". > >>> > >>>> Thanks. > >>>>>>> > >>>>>>> > >>>>>>> Mike Drob wrote: > >>>>>>> > >>>>>>> The proposed contribution is a collection of 11 examples. It's > >>>>>>> > >>>>>> clearly > >>> > >>>> non-trivial, which is probably enough to be considered "substantial" > >>>>>>>> > >>>>>>>> On Wed, Nov 12, 2014 at 12:58 PM, Josh Elser > > >>>>>>>> wrote: > >>>>>>>> > >>>>>>>> > >>>>>>>> Sean Busbey wrote: > >>>>>>>> > >>>>>>>>> On Wed, Nov 12, 2014 at 12:31 PM, Josh Elser< > >>>>>>>>> > >>>>>>>> josh.elser@gmail.com> > >>> > >>>> wrote: > >>>>>>>>>> > >>>>>>>>>> Personally, I didn't really think that this contribution was > >>>>>>>>>> in > >>>>>>>>>> > >>>>>>>>> the > >>>> > >>>>> spirit > >>>>>>>>>> > >>>>>>>>>>> of what the new codebase adoption guidelines were meant to > cover. > >>>>>>>>>>> > >>>>>>>>>>> Some extra examples which leverage what Accumulo already does > >>>>>>>>>>> > >>>>>>>>>> seems > >>> > >>>> more > >>>>>>>>>>> like improvements for new Accumulo users than anything else. > >>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>>> It's content developed out side of the project list. That's > >>>>>>>>>>> > >>>>>>>>>> all > >>> > >>>> it > >>>> > >>>>> takes to > >>>>>>>>>>> > >>>>>>>>>> require the trip through the Incubator checks as far as the ASF > >>>>>>>>>> guidelines > >>>>>>>>>> are concerned. > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> From http://incubator.apache.org/ip-clearance/index.html > >>>>>>>>>> > >>>>>>>>>> """ > >>>>>>>>> From time to time, an external codebase is brought into the > ASF > >>>>>>>>> > >>>>>>>> that > >>>> > >>>>> is > >>>>>>>>> not a separate incubating project but still represents a > >>>>>>>>> > >>>>>>>> substantial > >>> > >>>> contribution that was not developed within the ASF's source control > >>>>>>>>> system > >>>>>>>>> and on our public mailing lists. > >>>>>>>>> """ > >>>>>>>>> > >>>>>>>>> Not to look a gift-horse in the mouth (it is great work), but I > >>>>>>>>> > >>>>>>>> don't > >>> > >>>> see > >>>>>>>>> these examples as "substantial". I haven't found guidelines yet > >>>>>>>>> > >>>>>>>> that > >>> > >>>> better > >>>>>>>>> clarify the definition of "substantial". > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> > >> > --089e013a1ce8539fe20507d63593--