On Wed, Jan 22, 2020 at 02:09:39PM +0900, Michael Paquier wrote:
On Tue, Jan 21, 2020 at 05:20:17PM +0100, Tomas Vondra wrote:
Yeah, you're right returning them with feedback seems more appropriate,
given the long inactivity. Plus, the CF app apparently does not allow
moving WoA patches to the next CF anyway.
FWIW, I tend to take a base of two weeks as a sensible period of
time as that's half the CF period when I do the classification job.
Yeah. I've only nagged about patches that have been set to WoA before
the CF began, so far.
Those are the patches that have been set as WoA before this CF, and have
not been updated since. It's quite possible the state is stale for some
of those patches, although I've tried to check if there were any
messages on the list.
You need to be careful about bug fixes, as these are things that we
don't want to lose track of. Another thing that I noticed in the past
is that some patches are registered as bug fixes, but they actually
implement a new feature. So there can be tricky cases.
--
Makes sense.
--
Tomas Vondra http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
