Skip site navigation (1) Skip section navigation (2)

CommitFest reviews and application support

From: Dimitri Fontaine <dfontaine(at)hi-media(dot)com>
To: pgsql-hackers(at)postgresql(dot)org
Subject: CommitFest reviews and application support
Date: 2009-08-10 07:45:34
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgsql-hackers

This commitfest will soon finish and we can already say, I think, that
the support software is doing a pretty good job helping through
it. Congrats!

Now after some discussion about it on IRC, we have some ideas to improve
the situation some more. Specifically, reviews are touching several
areas and all reviewers aren't comfortable in all of them (it's about
skills, time, or perf testing lab equipment, mainly). What about
refining what parts of reviewing a specific review touches? Here's a
first try at a list of those:

 - patch (applies, merge, compiles, pass regression)
 - code reading (looks like it was already there, no WTF?) [1]
 - documentation (covers code, targets users, is sufficient)
 - testing (code behavior is what is documented, works well)
 - creative testing (tried hard to crash it)
 - perf testing (profiling, no regression in non optimized cases...)
 - you name it

On the application side of things, having checkboxes to check at review
submit and little icons on the "action lines" in general view could do

When we have that, some reviewers could concentrate on code review only
and waits for the patch to have been reviewed first for the other points
(patch, documentation, testing, e.g.). And reviewers feeling they won't
help reviewing code would omit checking this particular box.

Ideas, comments?



pgsql-hackers by date

Next:From: Christian ThomsenDate: 2009-08-10 08:41:05
Subject: Segmentation fault when using a set-returning C function from a view in 8.4.0
Previous:From: Peter EisentrautDate: 2009-08-10 07:42:34
Subject: Re: GRANT ON ALL IN schema

Privacy Policy | About PostgreSQL
Copyright © 1996-2017 The PostgreSQL Global Development Group