Re: [Ksummit-discuss] [CORE TOPIC] Recruitment (Reviewers, Testers, Maintainers, Hobbyists)

[Mark Brown <broonie@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1506 bytes --]

On Sat, Jul 18, 2015 at 08:19:14AM -0400, Steven Rostedt wrote:

> One ping a week is just a reminder. It doesn't fill your mailbox
> quickly. It doesn't mean you need to do it in that week. Also, the
> quicker it goes in before the merge window, the more testing it will
> receive.

It adds up pretty quickly.  No argument that merging things quickly is
good but having to read more content free e-mails is not going to help
with that.  Indeed if I'm expecting someone else to review a patch for
some reason (eg, driver maintainers) a week is about the normal time I'd
give for them to ack it if I'm confident in the patch myself so it's
sometimes a deliberately introduced delay.

Like I say my experience as a patch submitter is that I'd be sending
pings more often than not at a week whereas it's reasonably infrequent
that I need to resend anything.

> Really, a ping once a week will bother you? The once a week was what I
> and a few others started to enforce when we were getting pings once a
> day.

Yes, it does.  It's far too close to the window of normal delays that
happen (finding time to read larger or more complicated patches and
serieses, travel, whatever) and my guess is that my response if everyone
was nagging at that rate would be to treat the nags as noise and ignore
them which defeats the purpose.  These things only work if they're
unusual.

Two weeks feels like a better lower bound to me for the point between
people being "normally" busy and there being some kind of problem.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]