Thread: September Commit Fest coming soon!

September Commit Fest coming soon!

From
Josh Berkus
Date:
Folks,

The september commit fest starts in one week.  The goal, this time, is 
to start reviewing on day 1 of the commit fest and not spend the first 3 
days collecting extra patches.

So if your patch isn't on the list *on* September 1, it may get pushed 
into November.  So get it on the list now!

http://wiki.postgresql.org/wiki/CommitFest:2008-09

Second, I need to know who's available for round-robin reviewing for 
this commitfest (September 1-15).  If you volunteer, you'll be assigned 
a patch to review on September 4 from the list of unclaimed patches, and 
will be expected to review it within 5 days.  And then hopefully be 
assigned a second one.  Please, we really need help reviewing, and this 
is a good way to gain experience.

--Josh "Commitfest Mom" Berkus


Re: September Commit Fest coming soon!

From
"Jaime Casanova"
Date:
On Tue, Aug 26, 2008 at 1:28 PM, Josh Berkus <josh@agliodbs.com> wrote:
>
> Second, I need to know who's available for round-robin reviewing for this
> commitfest (September 1-15).

let's make a try... count with me for round-robin reviewing...

--
regards,
Jaime Casanova
Soporte y capacitación de PostgreSQL
Asesoría y desarrollo de sistemas
Guayaquil - Ecuador
Cel. (593) 87171157


Re: September Commit Fest coming soon!

From
"Hitoshi Harada"
Date:
Hi,

2008/8/27 Josh Berkus <josh@agliodbs.com>:
> Folks,
>
> The september commit fest starts in one week.  The goal, this time, is to
> start reviewing on day 1 of the commit fest and not spend the first 3 days
> collecting extra patches.
>
> So if your patch isn't on the list *on* September 1, it may get pushed into
> November.  So get it on the list now!
>
> http://wiki.postgresql.org/wiki/CommitFest:2008-09
>

For the git hosted projects, should we send in the latest patch file
to this list? Or just keep it static and reviewers will extract diff
from there?

-- 
Hitoshi Harada


Re: September Commit Fest coming soon!

From
Abhijit Menon-Sen
Date:
At 2008-08-28 20:09:21 +0900, umi.tanuki@gmail.com wrote:
>
> For the git hosted projects, should we send in the latest patch file
> to this list?

Yes, please do that.

-- ams


Re: September Commit Fest coming soon!

From
Tom Lane
Date:
Abhijit Menon-Sen <ams@oryx.com> writes:
> At 2008-08-28 20:09:21 +0900, umi.tanuki@gmail.com wrote:
>> For the git hosted projects, should we send in the latest patch file
>> to this list?

> Yes, please do that.

Yes.  You should *not* assume that reviewers are going to look at the
git repository.
        regards, tom lane


Re: September Commit Fest coming soon!

From
Tom Lane
Date:
Gregory Stark <stark@enterprisedb.com> writes:
> Tom Lane <tgl@sss.pgh.pa.us> writes:
>> Yes.  You should *not* assume that reviewers are going to look at the
>> git repository.

> However when a committer is actually looking to commit the code he'll want to
> be sure he has the most recent snapshot including any later changes that might
> not have warranted an email.

I'm going to state for the record that I have no intention of digging
around in unofficial repositories for unsubmitted changes (that might
or might not have been intended to go in anyway).  If you don't like
what I commit, send a delta patch.
        regards, tom lane


Re: September Commit Fest coming soon!

From
Gregory Stark
Date:
Tom Lane <tgl@sss.pgh.pa.us> writes:

> Abhijit Menon-Sen <ams@oryx.com> writes:
>> At 2008-08-28 20:09:21 +0900, umi.tanuki@gmail.com wrote:
>>> For the git hosted projects, should we send in the latest patch file
>>> to this list?
>
>> Yes, please do that.
>
> Yes.  You should *not* assume that reviewers are going to look at the
> git repository.

I think we should include both. They serve two different purposes. The GIT
repository might disappear some day or be inaccessible for some users. Also
you might be asking for commentary on specific questions about that specific
snapshot.

However when a committer is actually looking to commit the code he'll want to
be sure he has the most recent snapshot including any later changes that might
not have warranted an email.

--  Gregory Stark EnterpriseDB          http://www.enterprisedb.com Ask me about EnterpriseDB's RemoteDBA services!