From: | Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Bruce Momjian <bruce(at)momjian(dot)us>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Re: [COMMITTERS] pgsql: pg_upgrade: Convert old visibility map format to new format. |
Date: | 2016-04-27 01:11:32 |
Message-ID: | CAD21AoCAteFiMnBeZ8Et7K16Mzbqzmm23u42Voc893-gCsH8FQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Wed, Apr 27, 2016 at 10:00 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Tue, Apr 26, 2016 at 8:45 PM, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
>> On Fri, Mar 11, 2016 at 05:36:34PM +0000, Robert Haas wrote:
>>> pg_upgrade: Convert old visibility map format to new format.
>>>
>>> Commit a892234f830e832110f63fc0a2afce2fb21d1584 added a second bit per
>>> page to the visibility map, but pg_upgrade has been unaware of it up
>>> until now. Therefore, a pg_upgrade from an earlier major release of
>>> PostgreSQL to any commit preceding this one and following the one
>>> mentioned above would result in invalid visibility map contents on the
>>> new cluster, very possibly leading to data corruption. This plugs
>>> that hole.
>>>
>>> Masahiko Sawada, reviewed by Jeff Janes, Bruce Momjian, Simon Riggs,
>>> Michael Paquier, Andres Freund, me, and others.
>>
>> I have tested the current git trees of all supported versions of
>> Postgres in all possible pg_upgrade combinations and they all worked
>> perfectly.
Thank you for testing!
Good news for me.
>
> That's good!
>
> All the commits related to this topic could use extra-careful review
> and testing. If Masahiko-san got anything wrong, or I did, this could
> eat people's data in ways that are very hard to track down.
>
Yeah, we should test this feature in various ways, and I'm going to do so.
Regards,
--
Masahiko Sawada
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2016-04-27 01:37:45 | pgsql: pg_dump: Message style improvements |
Previous Message | Robert Haas | 2016-04-27 01:00:38 | Re: [COMMITTERS] pgsql: pg_upgrade: Convert old visibility map format to new format. |
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2016-04-27 01:14:20 | Re: EXPLAIN VERBOSE with parallel Aggregate |
Previous Message | Bruce Momjian | 2016-04-27 01:10:46 | Re: Background Processes and reporting |