Re: [HACKERS] Please claim review items for commit fest!
On 7/2/08, Josh Berkus [EMAIL PROTECTED] wrote: Just in case anyone was unclear, this is how we're trying things for this commitfest: 1) Starting RIGHT NOW, reviewers should claim review items they are interested in or specially qualified to review. 2) This weekend, I will check for all items which don't have one or more reviewers and parcel them out to the Round Robin Reviewers who don't already have patches to review. You do not have to be a committer to be a reviewer. Anyone who knows C code and is familiar with PostgreSQL can be a reviewer. Heck, even non-C coders can review proposed APIs. Each item can have several reviewers, and probably should. Oh, also reviewers -- please try to use constructive criticism! Some people are submitting their first patch, and we don't want them to leave the project forever. Thanks! I don't understand one aspect - if I'm unfamiliar with Postgres and cannot do full review or am familiar but cannot do full review due to time aspects but still want to throw some quick comments, should I register on wiki? And potentially make some actual reviewers to skip the patch? -- marko -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Please claim review items for commit fest!
On Wed, Jul 2, 2008 at 11:37 AM, Marko Kreen [EMAIL PROTECTED] wrote: I don't understand one aspect - if I'm unfamiliar with Postgres and cannot do full review or am familiar but cannot do full review due to time aspects but still want to throw some quick comments, should I register on wiki? And potentially make some actual reviewers to skip the patch? In that situation, just add your comments to the wiki page using the appropriate template, but don't bother to list yourself as a reviewer (for the very reason you suggest). -- Dave Page EnterpriseDB UK: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Please claim review items for commit fest!
On 7/2/08, Dave Page [EMAIL PROTECTED] wrote: On Wed, Jul 2, 2008 at 11:37 AM, Marko Kreen [EMAIL PROTECTED] wrote: I don't understand one aspect - if I'm unfamiliar with Postgres and cannot do full review or am familiar but cannot do full review due to time aspects but still want to throw some quick comments, should I register on wiki? And potentially make some actual reviewers to skip the patch? In that situation, just add your comments to the wiki page using the appropriate template, but don't bother to list yourself as a reviewer (for the very reason you suggest). The comments should go to wiki? Not mailing list? -- marko -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Please claim review items for commit fest!
On Wed, Jul 2, 2008 at 11:44 AM, Marko Kreen [EMAIL PROTECTED] wrote: On 7/2/08, Dave Page [EMAIL PROTECTED] wrote: On Wed, Jul 2, 2008 at 11:37 AM, Marko Kreen [EMAIL PROTECTED] wrote: I don't understand one aspect - if I'm unfamiliar with Postgres and cannot do full review or am familiar but cannot do full review due to time aspects but still want to throw some quick comments, should I register on wiki? And potentially make some actual reviewers to skip the patch? In that situation, just add your comments to the wiki page using the appropriate template, but don't bother to list yourself as a reviewer (for the very reason you suggest). The comments should go to wiki? Not mailing list? It's a fine line (and slightly bendy line) - but simple comments can go on the wiki, discussion should go to the list and be referenced from the wiki. For example, see the 'returned for feedback' section at the end of the last commit fest: http://wiki.postgresql.org/wiki/CommitFest:2008-05 -- Dave Page EnterpriseDB UK: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Please claim review items for commit fest!
Dave Page [EMAIL PROTECTED] writes: On Wed, Jul 2, 2008 at 11:44 AM, Marko Kreen [EMAIL PROTECTED] wrote: The comments should go to wiki? Not mailing list? It's a fine line (and slightly bendy line) - but simple comments can go on the wiki, discussion should go to the list and be referenced from the wiki. For example, see the 'returned for feedback' section at the end of the last commit fest: http://wiki.postgresql.org/wiki/CommitFest:2008-05 It is a fine line, but I think anything about the substance of the patch really ought to go to the list so other people get a chance to respond. IMHO the wiki is best thought of as a kind of group todo list. Notes about the status of a patch and a bottom-line summary for future reference makes sense to keep there. Something like review found problems with memory management so we can reprioritize it without rereading the emails for every item. -- Gregory Stark EnterpriseDB http://www.enterprisedb.com Ask me about EnterpriseDB's 24x7 Postgres support! -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers