On 15 December 2014 at 19:52, Josh Berkus <josh@agliodbs.com> wrote:
> On 12/15/2014 11:27 AM, Robert Haas wrote:
>> I feel like we used to be better at encouraging people to participate
>> in the CF even if they were not experts, and to do the best they can
>> based on what they did know. That was a helpful dynamic. Sure, the
>> reviews weren't perfect, but more people helped, and reviewing some of
>> the patch well and some of it in a more cursory manner is way better
>> than reviewing none of it at all.
>
> Well, it was strongly expressed to me by a number of senior contributors
> on this list and at the developer meeting that inexpert reviews were not
> really wanted, needed or helpful. As such, I stopped recruiting new
> reviewers (and, for that matter, doing them myself). I don't know if
> the same goes for anyone else.
I don't remember saying that, hearing it or agreeing with it and I
don't agree with it now.
-- Simon Riggs http://www.2ndQuadrant.com/PostgreSQL Development, 24x7 Support, Training & Services