2009/10/15 Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>:
> KaiGai Kohei wrote:
>> I have to focus on my patches with highest priority in CommitFest,
>> but it may be possible to help reviewing the proposed patches in
>> the off-fest season. It is illegal/undesirable for the process?
>
> No, that's absolutely fine. During commitfests patch review is needed
> the most, but please do help whenever you have the time.
I agree. I would also mention that reviewing a patch can often be
done in a few hours, and there are certainly periods of downtime
during a CommitFest when your own patches won't need attention. Of
course, a major patch like Hot Standby needs a lot more reviewing, but
that's an exceptional case, and there's really no need for you to bite
off something that ambitious. Just reviewing 1 or 2 small patches per
CommitFest would be much appreciated, if it's something you can
manage.
Reviewing patches at other times is very helpful too. Many times
patch authors complain about not getting feedback between CommitFests,
so anything you can pick up and give feedback on helps move the
project forward (and also decreases the amount that has to get done
during the next CommitFest).
Thanks,
...Robert