Re: [HACKERS] Large object corruption during 'piped' pg_restore

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Bosco Rama <postgres(at)boscorama(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-general(at)postgresql(dot)org, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [HACKERS] Large object corruption during 'piped' pg_restore
Date: 2011-01-21 17:54:53
Message-ID: AANLkTimOBqtnTTK22vdrXmcX3BH7Dyc+koeDrVuQ9mBW@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-general pgsql-hackers

On Fri, Jan 21, 2011 at 12:44 PM, Bosco Rama <postgres(at)boscorama(dot)com> wrote:
> Tom Lane wrote:
>>
>> So I'm not sure whether to fix it, or leave it as a known failure case
>> in old branches.  Comments?
>
> I understand the reluctance to fool with stable code.  I have zero insight
> into your installed versions distribution and backward compatibility needs
> so any comment I may have here is purely selfish.
>
> As an end user there is one area of the DB that I want to work correctly
> 100% of the time and that is the dump/restore tool(s).  If it's not going
> to work under certain circumstances it should at least tell me so and
> fail.  I don't think having the tool appear to work while corrupting the
> data (even if documented as doing so) is a viable method of operation.

Yeah, I lean toward saying we should back-patch this. Yeah, it'll be
lightly tested, but it's a pretty confined change, so it's unlikely to
break anything else. ISTM the worst case scenario is that it takes
two minor releases to get it right, and even that seems fairly
unlikely.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-general by date

  From Date Subject
Next Message Daniele Varrazzo 2011-01-21 18:36:53 Constraint exclusion with box and integer
Previous Message Bosco Rama 2011-01-21 17:44:21 Re: Large object corruption during 'piped' pg_restore

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2011-01-21 18:03:22 Re: Sync Rep for 2011CF1
Previous Message Bruce Momjian 2011-01-21 17:50:47 Re: Moving test_fsync to /contrib?