Re: Materialized View patch broke pg_dump - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: Materialized View patch broke pg_dump
Date
Msg-id 513E0D72.2090605@dunslane.net
Whole thread Raw
In response to Re: Materialized View patch broke pg_dump  (Fujii Masao <masao.fujii@gmail.com>)
Responses Re: Materialized View patch broke pg_dump  (Kevin Grittner <kgrittn@ymail.com>)
List pgsql-hackers
On 03/11/2013 12:30 PM, Fujii Masao wrote:
> On Tue, Mar 12, 2013 at 12:43 AM, Andrew Dunstan <andrew@dunslane.net> wrote:
>> On 03/11/2013 10:43 AM, Andrew Dunstan wrote:
>>>
>>> On 03/06/2013 10:55 AM, Kevin Grittner wrote:
>>>> Bernd Helmle <mailings@oopsware.de> wrote:
>>>>
>>>>> Looking into this issue, it seems the version check in getTables() of
>>>>> pg_dump.c
>>>>> is wrong. Shouldn't the check be
>>>>>
>>>>> if (fout->remoteVersion >= 90300)
>>>>> {
>>>>>
>>>>> }
>>>>>
>>>>> since this is where pg_relation_is_scannable() is introduced?
>>>> Fixed.
>>>>
>>>> Thanks for the report!
>>>>
>>>>
>>>
>>>
>>> I noticed this morning that I am still getting failures on 9.0, 9.1 and
>>> 9.2 which cause my cross-version upgrade testing to fail for git tip. For
>>> all I know this might apply to all back branches, but these are the only
>>> ones tested for upgrade, so that's all I can report on reliably.
>>>
>>> I'm chasing it up to find out exactly what's going on, but figured some
>>> extra eyeballs would help.
>>>
>>>
>> The problem is that pg_dump is sending an empty query in versions less than
>> 9.3, and choking on that. Suggested fix attached - there's really no reason
>> to be doing anything re mat views in versions < 9.3.
> This is the same problem that I reported in another thread.
> http://www.postgresql.org/message-id/CAHGQGwH+4vtyq==L6HRuPxTggfqrnLf0mWj75BfisOske28gMA@mail.gmail.com
>
>

Oh, I missed that. Yes, either of these would work.

cheers

andrew



pgsql-hackers by date:

Previous
From: Josh Berkus
Date:
Subject: Re: transforms
Next
From: Tom Lane
Date:
Subject: Re: postgres_fdw vs data formatting GUCs (was Re: [v9.3] writable foreign tables)