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 7B105F2E8 for ; Fri, 14 Nov 2014 16:27:49 +0000 (UTC) Received: (qmail 15506 invoked by uid 500); 14 Nov 2014 16:27:49 -0000 Delivered-To: apmail-accumulo-dev-archive@accumulo.apache.org Received: (qmail 15462 invoked by uid 500); 14 Nov 2014 16:27:49 -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 15451 invoked by uid 99); 14 Nov 2014 16:27:49 -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 16:27:49 +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.214.171 as permitted sender) Received: from [209.85.214.171] (HELO mail-ob0-f171.google.com) (209.85.214.171) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 14 Nov 2014 16:27:22 +0000 Received: by mail-ob0-f171.google.com with SMTP id wp18so12922862obc.16 for ; Fri, 14 Nov 2014 08:25:51 -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:sender:in-reply-to:references:from :date:message-id:subject:to:content-type; bh=yyPGss+9xKERtiXMpVUUN5Nw6ic2RuJqwSGqDLS9Hbo=; b=DszC0pEvp9IYTSDSPkh+mLizlCeZ7nzDrOy9QyuaDKMzolqdaBciP5IY14b1Vhqiw8 zfEZTk1puPNhhEJHAGYkn+01WALaez7Mzu6mju1vsuQkvL/g5BAHfLP1/kH/X0w7SDTd KWBpVbPYMSBSoeIp8X9WCRHoyWLgUFr61b8zK7zQqM19on1UyoHqWHHlZBXRrO/1N1D/ 9ncSsQHlDWjt1n/N/mOkx/3nWWpwOAWZjh5xcmEY+9OxYkO2qB+u7zlwF74Ncv4ShzI2 I8BXRRDsQcTSPnTzuEGwNKN1Zwf+Wo2LfyBtI44AeG6k1uLZYwMRDWt2KT8Oyu1jQcj/ bFMw== X-Gm-Message-State: ALoCoQnW1OHTS2LO7FBHVggB4zIWx2WlqYn1bce5GIMhgyg4lGCq+TdYBOKml6LFqhImyCZ/m2yK X-Received: by 10.202.51.69 with SMTP id z66mr8009416oiz.57.1415982350848; Fri, 14 Nov 2014 08:25:50 -0800 (PST) MIME-Version: 1.0 Sender: mdrob@cloudera.com Received: by 10.60.156.74 with HTTP; Fri, 14 Nov 2014 08:25:30 -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> From: Mike Drob Date: Fri, 14 Nov 2014 10:25:30 -0600 X-Google-Sender-Auth: vNuQuX9SQT18jpjHEbcsVLZsX3o Message-ID: Subject: Re: Contribute Examples/Exercises To: Accumulo Dev List Content-Type: multipart/alternative; boundary=001a113ce67407ee580507d41535 X-Virus-Checked: Checked by ClamAV on apache.org --001a113ce67407ee580507d41535 Content-Type: text/plain; charset=UTF-8 +1 On Fri, Nov 14, 2014 at 10:16 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". > > >>>>> > > >>>>> > > >>>>> > > >> > > > --001a113ce67407ee580507d41535--