From: | Craig Ringer <craig(at)2ndquadrant(dot)com> |
---|---|
To: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | CF app and patch series |
Date: | 2016-09-05 10:43:51 |
Message-ID: | CAMsr+YErMBOZkxq+dbg8Ls2NSuUGFXBVo3_p++ukLx3+HW1SmA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi all
Now that it's becoming more common to post patch series, not just
standalone patches, it might be worth looking at how the CF app can
help manage them.
Any ideas? Especially since the patch series may not get committed all
in one go, but progressively rebased on top of the bits that did get
committed until it's all in?
Making one CF app entry for this doesn't work well. It's confusing,
spammy in terms of number of entries involved, confusing for potential
reviewers, and pretty useless unless each patch is independent of all
the others. In which case they shouldn't be a series in the first
place.
A "partly committed" status doesn't help much; it doesn't signal
whether the rest is waiting for author response, awaiting review after
revision, etc.
So far I've just not been flagging it committed at all until
everything is. Or creating new CF entries that are immediately set to
"committed" for each part that goes in. But neither seems ideal.
Ideas?
--
Craig Ringer http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Aleksander Alekseev | 2016-09-05 11:03:11 | Re: Patch: Implement failover on libpq connect level. |
Previous Message | Julien Rouhaud | 2016-09-05 10:00:28 | Re: Assert(LWLockHeldByMeInMode(lock, LW_EXCLUSIVE)) |