From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Noah Misch <noah(at)leadboat(dot)com> |
Cc: | Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Andres Freund <andres(at)2ndquadrant(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Re: Better way of dealing with pgstat wait timeout during buildfarm runs? |
Date: | 2015-01-19 00:02:54 |
Message-ID: | 20520.1421625774@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Noah Misch <noah(at)leadboat(dot)com> writes:
> On Sun, Jan 18, 2015 at 04:09:29PM -0500, Tom Lane wrote:
>> After looking at the code, the minimum-change alternative would be more or
>> less as attached: first, get rid of the long-obsolete proposition that
>> autovacuum workers need fresher-than-usual stats; second, allow
>> pgstat_vacuum_stat to accept stats that are moderately stale (the number
>> given below allows them to be up to 50 seconds old); and third, suppress
>> wait-timeout warnings when the call is from pgstat_vacuum_stat. The third
>> point is what we need to avoid unnecessary buildfarm failures. The second
>> point addresses the idea that we don't need to stress the stats collector
>> too much for this.
> Only #3 belongs in a minimum-change patch. #1 and #2 solve and/or create
> different problems and operate independently of #3. A patch covering #3 alone
> sounds attractive.
[ raised eyebrow... ] In your previous message, you were advocating *for*
a change that was more invasive than this one. Why the change of heart?
In particular, I thought we'd already agreed to point #1.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2015-01-19 00:15:11 | Re: [REVIEW] Re: Fix xpath() to return namespace definitions |
Previous Message | Michael Paquier | 2015-01-19 00:00:34 | Re: [REVIEW] Re: Fix xpath() to return namespace definitions |