git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 2/2] send-email: supply a --send-delay=1 by default

From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date
Mar 25, 2018, 22:01 UTC
Message-ID
<87605jyfvi.fsf@evledraar.gmail.com>
In-Reply-To
<20180325210132.GE74743@genre.crustytoothpaste.net>
On Sun, Mar 25 2018, brian m. carlson wrote:
Show 47 quoted lines
> On Sun, Mar 25, 2018 at 06:28:03PM +0000, Ævar Arnfjörð Bjarmason wrote:
>> The earlier change to add this option described the problem this
>> option is trying to solve.
>>
>> This turns it on by default with a value of 1 second, which'll
>> hopefully solve it, and if not user reports as well as the
>> X-Mailer-Send-Delay header should help debug it.
>>
>> I think the trade-off of slowing down E-Mail sending to turn this on
>> makes sense because:
>>
>>  * GMail is a really common client, git.git's own unique authors by
>>    %aE are ~30% @gmail.com, ~20% for linux.git. That's just patch
>>    submitters, my guess is this it's much more common among those who
>>    mostly read the list, and those users who aren't using mu4e / mutt
>>    etc. anyway.
>>
>>  * There's really no point in having this feature at all if it's not
>>    made the default, since the entire point is to be able to read a
>>    list like the git ML or the LKML and have patches from others show
>>    up in order.
>>
>>  * I don't think anyone's really sensitive to the sending part of
>>    send-email taking longer. You just choose "all" and then switch to
>>    another terminal while it does its thing if you have a huge series,
>>    and for 1-3 patches I doubt anyone would notice this anyway.
>
> I'm not sure that this is going to have the effect you want it to have.
> Let me give an example to demonstrate why.
>
> If I send a series to the list, in order for this to work, you need my
> SMTP server (Postfix) to essentially send mails slowly enough to
> vger.kernel.org (ZMailer) that it doesn't batch them when it sends them
> to GMail.  The problem is that with my mail server, due to filtering and
> such, already takes at least a second to accept, process, and relay
> submitted messages.  vger still batched them and delivered them back to
> me out of order.  This will be even worse with large series.
>
> You are also assuming that my mail server will not have batched them and
> delivered them out of order, which it might well do, since Postfix uses
> a connection cache to machines that don't do STARTTLS (which, much to my
> annoyance, vger doesn't offer).
>
> In short, I don't think this is going to be especially helpful because
> it won't change the status quo for a lot of senders.  You'd have to
> insert some significant delay in order to get the effect you desire, and
> even then things could still be delivered out of order.

Good point. I also see that (via git log --author=Ævar --grep='^\[PATCH ') that this series itself arrived out of order (0 -> 2 -> 1), but I don't know to what extent public-inbox itself might be batching things.

It would be interesting to get reports from other GMail users as to what order these mails were shown in, but I think as soon as they're replied to that info's gone, at least for 2/2, which is the potentially out of order one in this case.

In general I realize that this won't be a general solution that'll work in all cases. E.g. I have a local SMTP on my laptop, if I'm on a plane it wouldn't matter if the delay was 2 hours, it would be batched up locally and sent all at once.

I was hoping we could find some sweet spot where the systems along the way (common smtpd's, majordomo, public-inbox's git repo) would as a result get this right most of the time for the purposes of appeasing this really common mail client, but maybe that's not going to work out.

Previous: brian m. carlsonNext: Eric Wong
Message 4 of 14 in “send-email: impose a delay while sending to appease GMail”
  1. 0/2 send-email: impose a delay while sending to appease GMailÆvar Arnfjörð Bjarmason, Mar 25, 2018
  2. 2/2 send-email: supply a --send-delay=1 by defaultÆvar Arnfjörð Bjarmason, Mar 25, 2018
  3. brian m. carlsonMar 25, 2018
  4. Ævar Arnfjörð BjarmasonMar 25, 2018
  5. Eric WongMar 28, 2018
  6. Junio C HamanoMar 26, 2018
  7. Eric SunshineMar 26, 2018
  8. Junio C HamanoMar 26, 2018
  9. 1/2 send-email: add an option to impose delay sent E-MailsÆvar Arnfjörð Bjarmason, Mar 25, 2018
  10. send-email: add an option to impose delay sent E-MailsÆvar Arnfjörð Bjarmason, Aug 14, 2018
  11. Stefan BellerAug 14, 2018
  12. Eric WongAug 14, 2018
  13. Junio C HamanoAug 14, 2018
  14. Ævar Arnfjörð BjarmasonAug 14, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.