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 513DFB96.40805@dunslane.net
Whole thread Raw
In response to Re: Materialized View patch broke pg_dump  (Andrew Dunstan <andrew@dunslane.net>)
Responses Re: Materialized View patch broke pg_dump  (Fujii Masao <masao.fujii@gmail.com>)
List pgsql-hackers
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.

cheers

andrew


Attachment

pgsql-hackers by date:

Previous
From: Craig Ringer
Date:
Subject: Re: Reproducible "Bus error" in 9.2.3 during database dump restoration (Ubuntu Server 12.04 LTS)
Next
From: Tom Lane
Date:
Subject: Re: matview join view error