incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matthieu Riou" <>
Subject Re: Re: [PROPOSAL] Thrift
Date Fri, 01 Feb 2008 22:53:20 GMT
On Feb 1, 2008 2:32 PM, Mark Slee <> wrote:

> Following up on what David sent out yesterday, I think we're all on
> board with having a single committers pool based upon mutual trust and
> respect.
> However, dropping parts of the code feels counterproductive to me, as I
> think it might put up a perceived barrier to collaboration with the
> Thrift project. Alternate language bindings were one of the main things
> we were really hoping to see come from the open source community, and
> we'd like to make sure this continues to be a really approachable thing.
> We wouldn't want developers to feel like their changes to Thrift won't
> be accepted without an up front investment of time building reputation
> in the Thrift community.
> I think reverting and reapplying these changes creates another risk.
> Developers would have incentive to keep some language-specific changes
> to themselves and not share them, simply because it would be more
> efficient, not because they are philosophically opposed to open source
> or sharing their changes at all.
> And to be quite frank, it feels very counterproductive to me to remove
> code from the project with full a priori intention of putting it back
> in. We'd rather focus on keeping development open and moving forwards.
> Removing people's code from the project could send an insulting and
> negative message.

I tend to agree with your observation. There have been donations in the past
that weren't completely clear in term of IP at the time the incubation
started and that have been cleared later on. IP cleanup is actually one of
the main purpose of the incubator. So I don't think this is a roadblock for
incubation even though it definitely is one for an official Apache release
and you'll need to clear up the remaining issues *before* being able to

I would also advise the creation of a page somewhere listing the parts of
the code that have unclear IPs as well as their respective status for the
sake of transparency and to keep track of the evolution. If you already have
a good idea of what these parts are, maybe this could even be included in
the proposal?


> Cheers,
> Mark
> -----Original Message-----
> From: Martijn Dashorst []
> Sent: Wednesday, January 30, 2008 10:39 AM
> To:
> Subject: Re: Re: [PROPOSAL] Thrift
> Perhaps in the interest of code audit (which needs to be done) and
> community building, the code parts of the missing committers should be
> removed from the code drop prior to incubation start, and be
> re-introduced inside the incubating podling by providing patches through
> bugzilla?
> Martijn
> On 1/30/08, David Reiss <> wrote:
> >
> > > If there are people who have already proven their *merit* on the
> > > project that are not included on the initial list of committers then
> > > I think they should be.
> >
> > >  > In reality, many parts of the Thrift code base are already
> > > entirely  > owned by non-Facebook entities. The Cocoa, C#, Perl, and
> > > Smalltalk  > implementations for Thrift were all developed entirely
> > > outside of  > Facebook, and although Facebook still maintains the
> > > trunk, we defer  > review of all these patches to the developers
> > > working on those  > libraries.
> > >
> > > So are these people on the initial list of committers?
> >
> > Perl was contributed by Jake Luciani, who is a committer, but the
> > developers of the Cocoa, C#, and Smalltalk bindings are not.  These
> > bindings were submitted as a set of a few patches (or in some cases,
> > even a single large patch), and added to the tree without extensive
> > review because, quite frankly, no committers were qualified to review
> > them.  Because, as far as we know, the original contributors are the
> > only users of these bindings, we've just been blindly committing any
> > changes they make, so it would make sense for them to have commit
> > access to their parts of the project.  However, they have not been
> > active enough in the Thrift community to warrant the trust that comes
> > with full commit access.
> >
> > I would say that the requirements for a language-binding contributor
> > to become a committer would be some amount of activity on the mailing
> > list and either a significant review of the binding by a qualified
> > committer or a significant project using the binding.
> >
> > --David
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail:
> > For additional commands, e-mail:
> >
> >
> --
> Buy Wicket in Action: Apache Wicket 1.3.0 is
> released Get it now:
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message