From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andrew Bille <andrewbille@gmail.com>
Cc: Alvaro Herrera <alvherre@2ndquadrant.com>
Cc: exclusion@gmail.com
Cc: pgsql-bugs@lists.postgresql.org
Subject: Re: BUG #16703: pg-dump fails to process recursive view definition
Date: Fri, 06 Nov 2020 15:25:39 -0500
Message-ID: <1624928.1604694339@sss.pgh.pa.us> (raw)
In-Reply-To: <1571034.1604675132@sss.pgh.pa.us>
References: <16703-e348f58aab3cf6cc@postgresql.org>
<1438431.1604588428@sss.pgh.pa.us>
<1473888.1604590908@sss.pgh.pa.us>
<1477409.1604593229@sss.pgh.pa.us>
<CAJnzarze97y5Q4NTxpc5zXuDNM-82g+0c1q_9HwD5DuqcBm4mQ@mail.gmail.com>
<1571034.1604675132@sss.pgh.pa.us>
I wrote:
> Andrew Bille <andrewbille@gmail.com> writes:
>> pg_dump is also fails to process the view created by the following script
>> (excerpt from privileges.sql):
>> CREATE USER user1;
>> CREATE TABLE test (col1 varchar(10), col2 boolean);
>> SET SESSION AUTHORIZATION user1;
>> CREATE VIEW testv AS SELECT * FROM test;
> Hm, yeah, so more to do here. (Sure glad we found these issues before
> next week's releases, not after.)
After some off-list discussion with Alvaro, we're thinking the best way
forward is to revert the pg_dump and LOCK TABLE changes for now, and
try again after next week's releases. My proposal about
> (2) Make LOCK TABLE ONLY on a view not recurse to the view's dependencies.
seems like it needs wider discussion, and there's not really time to
get that done before the wrap. Having found these two bugs in the patch
set doesn't inspire confidence that there aren't others, too.
regards, tom lane
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-bugs@postgresql.org
Cc: tgl@sss.pgh.pa.us, andrewbille@gmail.com, alvherre@2ndquadrant.com, exclusion@gmail.com, pgsql-bugs@lists.postgresql.org
Subject: Re: BUG #16703: pg-dump fails to process recursive view definition
In-Reply-To: <1624928.1604694339@sss.pgh.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox