Re: mailing list archiver chewing patches

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Matteo Beccati <php(at)beccati(dot)com>
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, Alvaro Herrera <alvherre(at)commandprompt(dot)com>, Joe Conway <mail(at)joeconway(dot)com>, Dimitri Fontaine <dfontaine(at)hi-media(dot)com>, David Fetter <david(at)fetter(dot)org>, Aidan Van Dyk <aidan(at)highrise(dot)ca>, Dave Page <dpage(at)pgadmin(dot)org>, Abhijit Menon-Sen <ams(at)toroid(dot)org>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Tim Bunce <Tim(dot)Bunce(at)pobox(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: mailing list archiver chewing patches
Date: 2010-02-01 16:28:15
Message-ID: 21249.1265041695@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-www

Matteo Beccati <php(at)beccati(dot)com> writes:
> My main concern is that we'd need to overcomplicate the thread detection
> algorithm so that it better deals with delayed messages: as it currently
> works, the replies to a missing message get linked to the
> "grand-parent". Injecting the missing message afterwards will put it at
> the same level as its replies. If it happens only once in a while I
> guess we can live with it, but definitely not if it happens tens of
> times a day.

That's quite common unfortunately --- I think you're going to need to
deal with the case. Even getting a direct feed from the mail relays
wouldn't avoid it completely: consider cases like

* A sends a message
* B replies, cc'ing A and the list
* B's reply to list is delayed by greylisting
* A replies to B's reply (cc'ing list)
* A's reply goes through immediately
* B's reply shows up a bit later

That happens pretty frequently IME.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Cristian 2010-02-01 16:28:19 BUG #5305: Postgres service stops when closing Windows session
Previous Message Tom Lane 2010-02-01 16:23:30 Re: Re: [COMMITTERS] pgsql: Define INADDR_NONE on Solaris when it's missing.

Browse pgsql-www by date

  From Date Subject
Next Message Josh Kupershmidt 2010-02-01 16:29:48 Typos in Feature Matrix Details
Previous Message Greg Sabino Mullane 2010-02-01 14:33:11 Re: Versions RSS page is missing version(s)