Re: Review-Then-Commit

2009-11-16 Thread Doug Cutting
Niclas Hedhman wrote: I am curious (never worked in a RTC environment); Does that mean that people turn down offers of commit rights? Does it mean that less commit rights are offered? Does it mean that commit rights are offered to those that do reviews even if they don't write much code? No

Re: Review-Then-Commit

2009-11-13 Thread Doug Cutting
Justin Erenkrantz wrote: As I understood Owen's Intro to Hadoop talk at AC, Hadoop has changed their methodology lately to CTR and found it to work far better. (Duh.) -- justin Hadoop uses RTC. Doug - To unsubscribe,

Re: Review-Then-Commit

2009-11-13 Thread Doug Cutting
With RTC as I've practiced it, becoming a committer doesn't so much give a privilege as a duty: committers are expected to review and commit patches promptly. Doug - To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org

Re: Review-Then-Commit

2009-11-13 Thread Niclas Hedhman
gets committed regardless of whether they have a commit bit or not. +1 With RTC as I've practiced it, becoming a committer doesn't so much give a privilege as a duty: committers are expected to review and commit patches promptly. I am curious (never worked in a RTC environment); Does that mean

Re: Review-Then-Commit

2009-11-12 Thread Ian Boston
On 12 Nov 2009, at 03:16, Greg Stein wrote: Not a strong opinion, but I think that RTC hampers the free-flow of ideas, experimentation, evolution, and creativity. It is a damper on expressivity. You maneuver bureaucracy to get a change in. CTR is about making a change and discussing it. But

Re: Review-Then-Commit

2009-11-12 Thread Michael Wechner
Ian Boston schrieb: not least because committed mistakes demand fixing by the committer and then anyone who can fix the bug. The only downside is that occasionally trunk wont build/run and if trunk is close to production that probably matters. I think another downside is, that (maybe

Re: Review-Then-Commit

2009-11-12 Thread Emmanuel Lecharny
Michael Wechner wrote: Ian Boston schrieb: not least because committed mistakes demand fixing by the committer and then anyone who can fix the bug. The only downside is that occasionally trunk wont build/run and if trunk is close to production that probably matters. I think another

Re: Review-Then-Commit

2009-11-12 Thread Paul Querna
On Wed, Nov 11, 2009 at 7:16 PM, Greg Stein gst...@gmail.com wrote: Not a strong opinion, but I think that RTC hampers the free-flow of ideas, experimentation, evolution, and creativity. It is a damper on expressivity. You maneuver bureaucracy to get a change in. CTR is about making a change

Re: Review-Then-Commit

2009-11-12 Thread Bruce Snyder
On Wed, Nov 11, 2009 at 8:16 PM, Greg Stein gst...@gmail.com wrote: Not a strong opinion, but I think that RTC hampers the free-flow of ideas, experimentation, evolution, and creativity. It is a damper on expressivity. You maneuver bureaucracy to get a change in. CTR is about making a change

Re: Review-Then-Commit

2009-11-12 Thread Eric Evans
On Thu, 2009-11-12 at 08:44 +0100, Justin Erenkrantz wrote: I think part of Cassandra's problem is that they do releases directly from trunk and don't have a 'stable' et al branch. No, this isn't (has never been) true. https://svn.apache.org/repos/asf/incubator/cassandra/branches/ The

Re: Review-Then-Commit

2009-11-12 Thread Eric Evans
On Thu, 2009-11-12 at 07:16 +, ant elder wrote: so about 6 months ago to try to help with problems they were having, and since then 99% of the commits have been made by only two people. I assume you're referring to Jonathan Ellis and myself, and I'm not sure that's exactly fair. There are

Re: Review-Then-Commit

2009-11-12 Thread Eric Evans
On Wed, 2009-11-11 at 22:16 -0500, Greg Stein wrote: Not a strong opinion, but I think that RTC hampers the free-flow of ideas, experimentation, evolution, and creativity. It is a damper on expressivity. You maneuver bureaucracy to get a change in. CTR is about making a change and discussing

Re: Review-Then-Commit

2009-11-12 Thread Matthieu Riou
On Thu, Nov 12, 2009 at 8:24 AM, ant elder ant.el...@gmail.com wrote: On Thu, Nov 12, 2009 at 4:12 PM, Eric Evans eev...@rackspace.com wrote: On Thu, 2009-11-12 at 07:16 +, ant elder wrote: so about 6 months ago to try to help with problems they were having, and since then 99% of the

Re: Review-Then-Commit

2009-11-12 Thread Jonathan Ellis
On Thu, Nov 12, 2009 at 10:36 AM, Greg Stein gst...@gmail.com wrote: On Thu, Nov 12, 2009 at 11:32, Eric Evans eev...@rackspace.com wrote: I agree with you, but tabled my protest because in practice what we have is working, doesn't seem to be a barrier to contribution, and everyone seems happy

Re: Review-Then-Commit

2009-11-12 Thread Greg Stein
On Thu, Nov 12, 2009 at 11:32, Eric Evans eev...@rackspace.com wrote: ... I agree with this, and as a Cassandra committer I have in the past protested our use of RTC. However, the current work-flow *in practice* is more about having someone, anyone, give changes a once over (making sure they

Re: Review-Then-Commit

2009-11-12 Thread ant elder
On Thu, Nov 12, 2009 at 4:12 PM, Eric Evans eev...@rackspace.com wrote: On Thu, 2009-11-12 at 07:16 +, ant elder wrote: so about 6 months ago to try to help with problems they were having, and since then 99% of the commits have been made by only two people. I assume you're referring to

Re: Review-Then-Commit

2009-11-12 Thread Jonathan Ellis
On Thu, Nov 12, 2009 at 10:24 AM, ant elder ant.el...@gmail.com wrote: So about 40% of the committed code is coming from others and reviewed by others - great - why not make some of those others committers? It's a long tail sort of thing. We follow the convention Johan suggested of assigning

Re: Review-Then-Commit

2009-11-12 Thread Matthieu Riou
On Thu, Nov 12, 2009 at 8:36 AM, Greg Stein gst...@gmail.com wrote: On Thu, Nov 12, 2009 at 11:32, Eric Evans eev...@rackspace.com wrote: ... I agree with this, and as a Cassandra committer I have in the past protested our use of RTC. However, the current work-flow *in practice* is more

Re: Review-Then-Commit

2009-11-12 Thread ant elder
On Thu, Nov 12, 2009 at 4:36 PM, Jonathan Ellis jbel...@gmail.com wrote: On Thu, Nov 12, 2009 at 10:24 AM, ant elder ant.el...@gmail.com wrote: So about 40% of the committed code is coming from others and reviewed by others - great - why not make some of those others committers? It's a long

Re: Review-Then-Commit

2009-11-12 Thread Eric Evans
On Thu, 2009-11-12 at 11:36 -0500, Greg Stein wrote: I agree with you, but tabled my protest because in practice what we have is working, doesn't seem to be a barrier to contribution, and everyone seems happy with it (even the casual contributors). I wouldn't say everyone. This whole

Re: Review-Then-Commit

2009-11-12 Thread Matthieu Riou
On Thu, Nov 12, 2009 at 8:55 AM, ant elder antel...@apache.org wrote: On Thu, Nov 12, 2009 at 4:36 PM, Jonathan Ellis jbel...@gmail.com wrote: On Thu, Nov 12, 2009 at 10:24 AM, ant elder ant.el...@gmail.com wrote: So about 40% of the committed code is coming from others and reviewed by

Re: Review-Then-Commit

2009-11-12 Thread Branko Čibej
Eric Evans wrote: Sure, but the IPMC is in a position of power, and can impose it's will upon the project (including CTR vs. RTC), right? I have no clue whether the IPMC can impose such a decision. But I'm very, very certain that it should not even consider trying. It's better to ask the

Re: Review-Then-Commit

2009-11-12 Thread Greg Stein
On Thu, Nov 12, 2009 at 11:44, Matthieu Riou matthieu.r...@gmail.com wrote: On Thu, Nov 12, 2009 at 8:24 AM, ant elder ant.el...@gmail.com wrote: On Thu, Nov 12, 2009 at 4:12 PM, Eric Evans eev...@rackspace.com wrote: On Thu, 2009-11-12 at 07:16 +, ant elder wrote: so about 6 months ago

Re: Review-Then-Commit

2009-11-12 Thread Matthieu Riou
On Thu, Nov 12, 2009 at 11:01 AM, Greg Stein gst...@gmail.com wrote: On Thu, Nov 12, 2009 at 11:44, Matthieu Riou matthieu.r...@gmail.com wrote: On Thu, Nov 12, 2009 at 8:24 AM, ant elder ant.el...@gmail.com wrote: On Thu, Nov 12, 2009 at 4:12 PM, Eric Evans eev...@rackspace.com wrote:

Re: Review-Then-Commit

2009-11-12 Thread Aidan Skinner
On Thu, Nov 12, 2009 at 3:16 AM, Greg Stein gst...@gmail.com wrote: Not a strong opinion, but I think that RTC hampers the free-flow of ideas, experimentation, evolution, and creativity. It is a damper on expressivity. You maneuver bureaucracy to get a change in. CTR is about making a change

Re: Review-Then-Commit

2009-11-12 Thread Greg Stein
Yup. We have all had different experiences, and I certainly acknowledge it is possible to have a successful RTC model in place. The real problem is that there is always a success story for any position. See? It works here. And there are *so* many factors that go into that success, beyond the

Review-Then-Commit

2009-11-11 Thread Matthieu Riou
Hi guys, What's the take of other mentors and the IPMC on podlings practicing RTC? I'm asking because some seem to see it as a blocker for graduation whereas I see it much more as a development methodology with little community impact and therefore no real influence on graduation. Strong opinions

Re: Review-Then-Commit

2009-11-11 Thread Martijn Dashorst
As long as the community is not divided on the issue whether to practice RTC vs CTR, I see no blocker for graduation. That is: as long as RTC was not installed to mitigate problems inside the community. If that is the case, the community may still be broken, with the underlying issue mopped under

Re: Review-Then-Commit

2009-11-11 Thread Emmanuel Lecharny
Matthieu Riou wrote: Hi guys, What's the take of other mentors and the IPMC on podlings practicing RTC? I'm asking because some seem to see it as a blocker for graduation whereas I see it much more as a development methodology with little community impact and therefore no real influence on

Re: Review-Then-Commit

2009-11-11 Thread Daniel Kulp
As Martijn alluded to, I think we'd need some more context as to why and how they use RTC. If all the reviews are done by a single person because that is what they want, THAT would be a problem. If the reviews are a community driven thing and the community thinks that's the best way for

Re: Review-Then-Commit

2009-11-11 Thread Matthieu Riou
On Wed, Nov 11, 2009 at 8:31 AM, Daniel Kulp dk...@apache.org wrote: As Martijn alluded to, I think we'd need some more context as to why and how they use RTC. Yes, sorry for the lack of details. The context is Cassandra and they're doing RTC by community choice. They all seem to agree that

Re: Review-Then-Commit

2009-11-11 Thread Leo Simons
On Wed, Nov 11, 2009 at 5:20 PM, Matthieu Riou matth...@offthelip.org wrote: On Wed, Nov 11, 2009 at 8:31 AM, Daniel Kulp dk...@apache.org wrote: As Martijn alluded to, I think we'd need some more context as to why and how they use RTC. Yes, sorry for the lack of details. The context is

Re: Review-Then-Commit

2009-11-11 Thread Matthieu Riou
On Wed, Nov 11, 2009 at 4:05 PM, Leo Simons m...@leosimons.com wrote: On Wed, Nov 11, 2009 at 5:20 PM, Matthieu Riou matth...@offthelip.org wrote: On Wed, Nov 11, 2009 at 8:31 AM, Daniel Kulp dk...@apache.org wrote: As Martijn alluded to, I think we'd need some more context as to why and

Re: Review-Then-Commit

2009-11-11 Thread Bruce Snyder
On Wed, Nov 11, 2009 at 9:21 AM, Emmanuel Lecharny elecha...@apache.org wrote: Matthieu Riou wrote: Hi guys, What's the take of other mentors and the IPMC on podlings practicing RTC? I'm asking because some seem to see it as a blocker for graduation whereas I see it much more as a

Re: Review-Then-Commit

2009-11-11 Thread Greg Stein
Not a strong opinion, but I think that RTC hampers the free-flow of ideas, experimentation, evolution, and creativity. It is a damper on expressivity. You maneuver bureaucracy to get a change in. CTR is about making a change and discussing it. But you get *forward progress*. I also feel that RTC

Re: Review-Then-Commit

2009-11-11 Thread ant elder
On Wed, Nov 11, 2009 at 4:31 PM, Daniel Kulp dk...@apache.org wrote: As Martijn alluded to, I think we'd need some more context as to why and how they use RTC. This appears to be where it came from: http://markmail.org/message/d45dmasuwnda25wd so about 6 months ago to try to help with

Re: Review-Then-Commit

2009-11-11 Thread ant elder
I agree with that. And before graduation I think it might be worth trying to get CTR used more, they do seem open this - http://markmail.org/message/i255ekzxpuesow44 ...ant On Thu, Nov 12, 2009 at 3:16 AM, Greg Stein gst...@gmail.com wrote: Not a strong opinion, but I think that RTC hampers

Re: Review-Then-Commit

2009-11-11 Thread Justin Erenkrantz
On Thu, Nov 12, 2009 at 4:16 AM, Greg Stein gst...@gmail.com wrote: I've participated in both styles of development. RTC is *stifling*. I would never want to see that in any Apache community for its routine development (branch releases are another matter). My opinion is that it is very