Re: Proposal: Log inability to lock pages during vacuum

From: Jim Nasby <Jim(dot)Nasby(at)BlueTreble(dot)com>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, Greg Stark <stark(at)mit(dot)edu>, Andres Freund <andres(at)2ndquadrant(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Proposal: Log inability to lock pages during vacuum
Date: 2014-11-10 18:03:59
Message-ID: 5460FE0F.20402@BlueTreble.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 11/10/14, 11:28 AM, Alvaro Herrera wrote:
> Jim Nasby wrote:
>> On 11/7/14, 8:21 PM, Robert Haas wrote:
>>> On Thu, Nov 6, 2014 at 8:03 PM, Jim Nasby <Jim(dot)Nasby(at)bluetreble(dot)com> wrote:
>>>> The problem right now is there's no way to actually obtain evidence that
>>>> this is (or isn't) something to worry about, because we just silently skip
>>>> pages. If we had any kind of tracking on this we could stop guessing. :(
>>>
>>> I could see logging it, but I agree with Andres and Alvaro that the
>>> odds are strongly against there being any actual problem here.
>>
>> I'm fine with that. Any other objections? Andres?
>
> If what we want is to quantify the extent of the issue, would it be more
> convenient to save counters to pgstat? Vacuum already sends pgstat
> messages, so there's no additional traffic there.

IMHO that would be ideal, but I think Tom was leery of using more space for every table. If we go this route, I'm guessing we should only log pages we skip, and not log pages we had to wait for the lock on (in the case of a freeze). Also, should we still eroprt this even if we are putting it in stats?

Is there a way to avoid duplicating the entire eroprt call? I see I could call errstart & friends manually, but currently that's only done in elog.c.
--
Jim Nasby, Data Architect, Blue Treble Consulting
Data in Trouble? Get it in Treble! http://BlueTreble.com

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2014-11-10 18:05:46 Re: Allow signal handlers to optionally use SA_SIGINFO information?
Previous Message Simon Riggs 2014-11-10 18:00:22 Re: Teaching pg_dump to use NOT VALID constraints