On Fri, 2008-09-05 at 17:19 +0300, Marko Kreen wrote:
> >
> > I think this should be organised with different kinds of reviewer:
>
> The list is correct but too verbose. And it does not attack the core
> of the problem. I think the problem is not:
>
> What can/should I do?
>
> but instead:
>
> Can I take the responsibility?
Completely agree. The list was really an example of the different styles
of review that are possible, not a rigid categorisation that must be
followed.
-- Simon Riggs www.2ndQuadrant.comPostgreSQL Training, Services and Support