Re: [HACKERS] Re: PD_ALL_VISIBLE flag was incorrectly set happend during repeatable vacuum

From: Merlin Moncure <mmoncure(at)gmail(dot)com>
To: Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>
Cc: daveg <daveg(at)sonic(dot)net>, Robert Haas <robertmhaas(at)gmail(dot)com>, Greg Stark <gsstark(at)mit(dot)edu>, David Christensen <david(at)endpoint(dot)com>, Maxim Boguk <maxim(dot)boguk(at)gmail(dot)com>, Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, pgsql-admin(at)postgresql(dot)org, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [HACKERS] Re: PD_ALL_VISIBLE flag was incorrectly set happend during repeatable vacuum
Date: 2011-03-03 15:04:04
Message-ID: AANLkTi=wKkEX_gX9-XOFm2DP85HvEAAJc4zUtD_509MQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-admin pgsql-hackers

On Thu, Mar 3, 2011 at 2:16 AM, Heikki Linnakangas
<heikki(dot)linnakangas(at)enterprisedb(dot)com> wrote:
> On 03.03.2011 09:12, daveg wrote:
>>
>> Question: what would be the consequence of simply patching out the setting
>> of this flag? Assuming that the incorrect PD_ALL_VISIBLE flag is the only
>> problem (big assumption perhaps) then simply never setting it would at
>> least
>> avoid the possibility of returning wrong answers, presumably at some
>> performance cost. We possibly could live with that until we get a handle
>> on the real cause and fix.
>
> Yes. With that assumption.
>
> If you really want to do that, I would suggest the attached patch instead.
> This just disables the optimization in seqscans to trust it, so an
> incorrectly set flag won't affect correctness of query results, but the
> flag is still set as usual and you still get the warnings so that we can
> continue to debug the issue.

This. The mis-set flag can is likely a bug/concurrency issue etc,
but could also be a symptom of more sinister data corruption. I did
various vacuum experiments all day yesterday on my windows workstation
and was not able to produce any mis-flags. I trust iscsi more than
nfs, but maybe there is a connection here that is hardware based. hm.
do you think it would be helpful to know what is causing the
all_visible flag to get flipped? If so, the attached patch shows
which case is throwing it...

merlin

Attachment Content-Type Size
visible_debug.patch application/octet-stream 2.5 KB

In response to

Responses

Browse pgsql-admin by date

  From Date Subject
Next Message Jason Clark 2011-03-03 16:54:02 Re: HA warm standby issue with trigger file.
Previous Message Kenneth Marshall 2011-03-03 14:10:54 Re: long running commits

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2011-03-03 15:13:43 Mark deprecated operators as such in their comments?
Previous Message Fujii Masao 2011-03-03 15:02:43 Re: Sync Rep v19