| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
| Cc: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, pgsql-hackers(at)postgresql(dot)org, Amit Langote <amitlangote09(at)gmail(dot)com> |
| Subject: | Re: \d with triggers: more than one row returned by a subquery used as an expression |
| Date: | 2022-01-18 00:14:34 |
| Message-ID: | 163710.1642464874@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Justin Pryzby <pryzby(at)telsasoft(dot)com> writes:
> On Mon, Jan 17, 2022 at 05:02:00PM -0500, Tom Lane wrote:
>> ISTM the real problem is the assumption that only related triggers could
>> share a tgname, which evidently isn't true. I think this query needs to
>> actually match on tgparentid, rather than taking shortcuts.
> I don't think that should be needed - tgparentid should match
> pg_partition_ancestors().
Uh, what? tgparentid is a trigger OID, not a relation OID.
> Is there any reason why WITH ORDINALITY can't work ?
> This is passing the smoke test.
How hard did you try to break it? It still seems to me that
this can be fooled by an unrelated trigger with the same tgname.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jelte Fennema | 2022-01-18 00:35:36 | Re: [EXTERNAL] Re: PQcancel does not use tcp_user_timeout, connect_timeout and keepalive settings |
| Previous Message | Justin Pryzby | 2022-01-18 00:08:24 | Re: \d with triggers: more than one row returned by a subquery used as an expression |