On 26 September 2014 08:48, Heikki Linnakangas <hlinnakangas@vmware.com> wrote:
> But in many cases, lack of good documentation makes even reviewing the patch
> difficult. How do you determine if the patch works as intended, if you don't
> know what it's supposed to do?
Exactly.
Lack of review and lack of consensus are often caused by the author
not making the patch fully and genuinely accessible to peer review.
Don't say you're having problems getting buy in when you've done very
little to encourage that. Committing early is not the solution.
-- Simon Riggs http://www.2ndQuadrant.com/PostgreSQL Development, 24x7 Support, Training & Services