pg.ddx.io pgsql-bugs@postgresql.org mailing list archive
help / color / mirror / Atom feedBUG #17646: create rule named "_RETURN" will cause pg core
6+ messages / 3 participants
[nested] [flat]
Commits from this thread:
2022-10-17 b21615d1e9 65c1106d8c 4a41a069e7 e9377e3e53 2f26cec488 ecf4ce689a 797e313dc9 Reject non-ON-SELECT rules that are named "_RETURN".
2022-10-17 62b263bf77 99b6b705d4 eec3466118 2e3326929b 8c611602bd Guard against table-AM-less relations in planner.
* BUG #17646: create rule named "_RETURN" will cause pg core
@ 2022-10-17 12:03 PG Bug reporting form <noreply@postgresql.org>
2022-10-17 14:17 ` Re: BUG #17646: create rule named "_RETURN" will cause pg core Tom Lane <tgl@sss.pgh.pa.us>
2022-10-17 14:38 ` Re: BUG #17646: create rule named "_RETURN" will cause pg core Ilya Anfimov <ilan@tzirechnoy.com>
0 siblings, 2 replies; 6+ messages in thread
From: PG Bug reporting form @ 2022-10-17 12:03 UTC (permalink / raw)
To: pgsql-bugs@lists.postgresql.org; +Cc: liukui@kingbase.com.cn
The following bug has been logged on the website:
Bug reference: 17646
Logged by: kui liu
Email address: liukui@kingbase.com.cn
PostgreSQL version: 15.0
Operating system: linux
Description:
Hi, I encounter an error, it may be a bug, thx.
here is a test case
create table t (id integer);
create view v1 as select * from t;
create or replace rule "_RETURN" as on UPDATE to v1 do instead select * from
t;
select * from v1;
execute up sql, will get this error
server closed the connection unexpectedly
This probably means the server terminated abnormally
before or while processing the request.
The connection to the server was lost. Attempting reset: Failed.
The connection to the server was lost. Attempting reset: Failed.
!?>
^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: BUG #17646: create rule named "_RETURN" will cause pg core
2022-10-17 12:03 BUG #17646: create rule named "_RETURN" will cause pg core PG Bug reporting form <noreply@postgresql.org>
@ 2022-10-17 14:17 ` Tom Lane <tgl@sss.pgh.pa.us>
2022-10-17 14:33 ` Re: BUG #17646: create rule named "_RETURN" will cause pg core Ilya Anfimov <ilan@tzirechnoy.com>
1 sibling, 1 reply; 6+ messages in thread
From: Tom Lane @ 2022-10-17 14:17 UTC (permalink / raw)
To: liukui@kingbase.com.cn; +Cc: pgsql-bugs@lists.postgresql.org
PG Bug reporting form <noreply@postgresql.org> writes:
> here is a test case
> create table t (id integer);
> create view v1 as select * from t;
> create or replace rule "_RETURN" as on UPDATE to v1 do instead select * from
> t;
Meh. We should be preventing you from doing that.
(Although the core dump is also not great --- something should've noticed
the bogosity of the plan earlier than this.)
regards, tom lane
^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: BUG #17646: create rule named "_RETURN" will cause pg core
2022-10-17 12:03 BUG #17646: create rule named "_RETURN" will cause pg core PG Bug reporting form <noreply@postgresql.org>
2022-10-17 14:17 ` Re: BUG #17646: create rule named "_RETURN" will cause pg core Tom Lane <tgl@sss.pgh.pa.us>
@ 2022-10-17 14:33 ` Ilya Anfimov <ilan@tzirechnoy.com>
2022-10-17 14:41 ` Re: BUG #17646: create rule named "_RETURN" will cause pg core Tom Lane <tgl@sss.pgh.pa.us>
0 siblings, 1 reply; 6+ messages in thread
From: Ilya Anfimov @ 2022-10-17 14:33 UTC (permalink / raw)
To: pgsql-bugs@lists.postgresql.org
On Mon, Oct 17, 2022 at 10:17:55AM -0400, Tom Lane wrote:
> PG Bug reporting form <noreply@postgresql.org> writes:
> > here is a test case
>
> > create table t (id integer);
> > create view v1 as select * from t;
> > create or replace rule "_RETURN" as on UPDATE to v1 do instead select * from
> > t;
>
> Meh. We should be preventing you from doing that.
>
> (Although the core dump is also not great --- something should've noticed
> the bogosity of the plan earlier than this.)
Id doesn't look much of a plan bogosity,
but rather implicit deletion of the "_RETURN" on select rule.
Consider the behaviour of the direct drop rule:
ilan=*> create table t (id integer);
CREATE TABLE
ilan=*> create view v1 as select * from t;
CREATE VIEW
ilan=*> DROP RULE "_RETURN" ON v1;
ERROR: cannot drop rule _RETURN on view v1 because view v1 requires it
ПОДСКАЗКА: You can drop view v1 instead.
>
> regards, tom lane
>
^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: BUG #17646: create rule named "_RETURN" will cause pg core
2022-10-17 12:03 BUG #17646: create rule named "_RETURN" will cause pg core PG Bug reporting form <noreply@postgresql.org>
2022-10-17 14:17 ` Re: BUG #17646: create rule named "_RETURN" will cause pg core Tom Lane <tgl@sss.pgh.pa.us>
2022-10-17 14:33 ` Re: BUG #17646: create rule named "_RETURN" will cause pg core Ilya Anfimov <ilan@tzirechnoy.com>
@ 2022-10-17 14:41 ` Tom Lane <tgl@sss.pgh.pa.us>
2022-10-17 15:15 ` Re: BUG #17646: create rule named "_RETURN" will cause pg core Ilya Anfimov <ilan@tzirechnoy.com>
0 siblings, 1 reply; 6+ messages in thread
From: Tom Lane @ 2022-10-17 14:41 UTC (permalink / raw)
To: Ilya Anfimov <ilan@tzirechnoy.com>; +Cc: pgsql-bugs@lists.postgresql.org
Ilya Anfimov <ilan@tzirechnoy.com> writes:
> On Mon, Oct 17, 2022 at 10:17:55AM -0400, Tom Lane wrote:
>> (Although the core dump is also not great --- something should've noticed
>> the bogosity of the plan earlier than this.)
> Id doesn't look much of a plan bogosity,
> but rather implicit deletion of the "_RETURN" on select rule.
Right, but then since there's no applicable rule, we end up generating
a plan that tries to do a seqscan directly on the view relation,
which of course lacks storage. Something should notice that a little
sooner than segfaulting because of rel->rd_tableam being NULL --- it's
not like we don't have hundreds of other sanity checks for not-really-
supposed-to-happen catalog corruption.
In versions before v12, I get something like
regression=# select * from v1;
ERROR: could not open file "base/16384/49209": No such file or directory
which is a shade less bad, but still not great.
regards, tom lane
^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: BUG #17646: create rule named "_RETURN" will cause pg core
2022-10-17 12:03 BUG #17646: create rule named "_RETURN" will cause pg core PG Bug reporting form <noreply@postgresql.org>
2022-10-17 14:17 ` Re: BUG #17646: create rule named "_RETURN" will cause pg core Tom Lane <tgl@sss.pgh.pa.us>
2022-10-17 14:33 ` Re: BUG #17646: create rule named "_RETURN" will cause pg core Ilya Anfimov <ilan@tzirechnoy.com>
2022-10-17 14:41 ` Re: BUG #17646: create rule named "_RETURN" will cause pg core Tom Lane <tgl@sss.pgh.pa.us>
@ 2022-10-17 15:15 ` Ilya Anfimov <ilan@tzirechnoy.com>
0 siblings, 0 replies; 6+ messages in thread
From: Ilya Anfimov @ 2022-10-17 15:15 UTC (permalink / raw)
To: pgsql-bugs@lists.postgresql.org
On Mon, Oct 17, 2022 at 10:41:35AM -0400, Tom Lane wrote:
> Ilya Anfimov <ilan@tzirechnoy.com> writes:
> > On Mon, Oct 17, 2022 at 10:17:55AM -0400, Tom Lane wrote:
> >> (Although the core dump is also not great --- something should've noticed
> >> the bogosity of the plan earlier than this.)
>
> > Id doesn't look much of a plan bogosity,
> > but rather implicit deletion of the "_RETURN" on select rule.
>
> Right, but then since there's no applicable rule, we end up generating
> a plan that tries to do a seqscan directly on the view relation,
> which of course lacks storage. Something should notice that a little
> sooner than segfaulting because of rel->rd_tableam being NULL --- it's
> not like we don't have hundreds of other sanity checks for not-really-
> supposed-to-happen catalog corruption.
>
> In versions before v12, I get something like
>
> regression=# select * from v1;
> ERROR: could not open file "base/16384/49209": No such file or directory
>
> which is a shade less bad, but still not great.
Another solution would be to always create a real heap file and let
anyone dropped to that state select it, if he wishes.
btw, the documentation clearly says that it does so:
https://www.postgresql.org/docs/14/rules-views.html
<<In fact, there is essentially no difference between:
CREATE VIEW myview AS SELECT * FROM mytab;
compared against the two commands:
CREATE TABLE myview (same column list as mytab);
CREATE RULE "_RETURN" AS ON SELECT TO myview DO INSTEAD
SELECT * FROM mytab;
because this is exactly what the CREATE VIEW command does internally. >>
^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: BUG #17646: create rule named "_RETURN" will cause pg core
2022-10-17 12:03 BUG #17646: create rule named "_RETURN" will cause pg core PG Bug reporting form <noreply@postgresql.org>
@ 2022-10-17 14:38 ` Ilya Anfimov <ilan@tzirechnoy.com>
1 sibling, 0 replies; 6+ messages in thread
From: Ilya Anfimov @ 2022-10-17 14:38 UTC (permalink / raw)
To: pgsql-bugs@lists.postgresql.org
On Mon, Oct 17, 2022 at 12:03:15PM +0000, PG Bug reporting form wrote:
> The following bug has been logged on the website:
>
> Bug reference: 17646
> Logged by: kui liu
> Email address: liukui@kingbase.com.cn
> PostgreSQL version: 15.0
> Operating system: linux
> Description:
>
> Hi, I encounter an error, it may be a bug, thx.
>
> here is a test case
>
> create table t (id integer);
> create view v1 as select * from t;
> create or replace rule "_RETURN" as on UPDATE to v1 do instead select * from
> t;
> select * from v1;
>
> execute up sql, will get this error
>
> server closed the connection unexpectedly
> This probably means the server terminated abnormally
> before or while processing the request.
> The connection to the server was lost. Attempting reset: Failed.
> The connection to the server was lost. Attempting reset: Failed.
> !?>
btw, I'm confirming the bug for postgres from at least 8.4 to 14.
Version 11 and less instead of segfault prints someting like:
ilan=*# select * from v1;
ERROR: could not open file "base/24576/57373": No such file or directory
>
^ permalink raw reply [nested|flat] 6+ messages in thread
end of thread, other threads:[~2022-10-17 15:15 UTC | newest]
Thread overview: 6+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2022-10-17 12:03 BUG #17646: create rule named "_RETURN" will cause pg core PG Bug reporting form <noreply@postgresql.org>
2022-10-17 14:17 ` Tom Lane <tgl@sss.pgh.pa.us>
2022-10-17 14:33 ` Ilya Anfimov <ilan@tzirechnoy.com>
2022-10-17 14:41 ` Tom Lane <tgl@sss.pgh.pa.us>
2022-10-17 15:15 ` Ilya Anfimov <ilan@tzirechnoy.com>
2022-10-17 14:38 ` Ilya Anfimov <ilan@tzirechnoy.com>
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