Re: Autovacuum breakage from a734fd5d1

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Autovacuum breakage from a734fd5d1
Date: 2016-11-28 02:36:07
Message-ID: CAB7nPqSk56smm53s8OQZ3uqDHVmdZi=wsSKMj672wAE+B8PfQA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Nov 28, 2016 at 10:02 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Sun, Nov 27, 2016 at 5:45 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> So the problem seems to be confirmed to exist, but be of low probability
>> and low consequences, in back branches. I think we only need to fix it in
>> HEAD. The lock acquisition and status recheck that I proposed before
>> should be sufficient.
>
> Thanks for digging into this. I failed to notice while reviewing that
> the way we were printing the message had changed a bit in the new
> code, and I just totally overlooked the existing locking hazards.
> Oops.

Sorry for the late reply.

In order to reproduce the failure I have just inserted a manual
pg_usleep before looping through the list of orphan_oids, and after
dropping manually from another session a couple of orphaned temporary
tables I was able to see the failure. Attached is a proposal of patch.
--
Michael

Attachment Content-Type Size
autovac-orphan-failure.patch invalid/octet-stream 933 bytes

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2016-11-28 02:36:53 Re: Mail thread references in commits
Previous Message Tom Lane 2016-11-28 02:33:59 Re: Autovacuum breakage from a734fd5d1