Re: [bug fix] Suppress "autovacuum: found orphan temp table" message

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Andres Freund <andres(at)2ndquadrant(dot)com>, MauMau <maumau307(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [bug fix] Suppress "autovacuum: found orphan temp table" message
Date: 2014-07-22 14:17:15
Message-ID: 21899.1406038635@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> On Tue, Jul 22, 2014 at 6:23 AM, Andres Freund <andres(at)2ndquadrant(dot)com> wrote:
>> No. Just removing a warning isn't the way to solve this. If you want to
>> improve things you'll actually need to improve things not just stick
>> your head into the sand.

> I've studied this area of the code before, and I would actually
> proposed to fix this in the opposite way - namely, adopt the logic
> currently used for the wraparound case, which drops the temp table,
> even if wraparound is not an issue. The current code seems to be
> laboring under the impression that there's some benefit to keeping the
> temporary relation around, which, as far as I can see, there isn't.

FWIW, I agree with Andres on this. The right response is to drop the temp
table not complain about log spam. Or even more to the point, investigate
why it's there in the first place; perhaps there's an actual fixable bug
somewhere in there. But deciding that orphaned temp tables are normal
is *not* an improvement IMO.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2014-07-22 14:25:56 Re: [bug fix] Suppress "autovacuum: found orphan temp table" message
Previous Message Andres Freund 2014-07-22 14:12:10 Re: [bug fix] Suppress "autovacuum: found orphan temp table" message