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
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 |
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? |