On 12/12/14, 1:44 PM, Joshua D. Drake wrote:
1. We don't need a full-time engineer to manage a commitfest. We need a manager or PM.
I don't think that's actually true. The major points on this thread are that 1) we don't have enough capacity for doing reviews and 2) the CFM has no authority to enforce anything. I see no way that #2 can be addressed by a mere manager/PM. If three *very* senior community members (David, Josh and Robert) couldn't get this done there's no way a PM could. (Well, I suppose of Tom was standing behind them with a flaming sword it might work...) Even so, this still wouldn't address the real problem, which is lack of review capacity. FWIW, I faced the same problem at Enova: good, solid reviews were very important for maintaining the quality of the data and database code, yet were a constant source of pain and friction. And that was with people being paid to do them, as well as a very extensive set of unit tests. In other words, this isn't an easy problem to solve. One thing that I think would help is modifying the CF app to allow for multiple reviewers. IIRC I reviewed 4 or 5 patches but I didn't mark myself as reviewer of any of them because I don't feel I have enough knowledge to fulfill that role. -- Jim Nasby, Data Architect, Blue Treble Consulting Data in Trouble? Get it in Treble! http://BlueTreble.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
