| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Alexander Korotkov <aekorotkov(at)gmail(dot)com> |
| Cc: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Neil Chen <carpenter(dot)nail(dot)cz(at)gmail(dot)com>, Alexander Law <exclusion(at)gmail(dot)com>, PostgreSQL mailing lists <pgsql-bugs(at)lists(dot)postgresql(dot)org>, Paul A Jungwirth <pj(at)illuminatedcomputing(dot)com> |
| Subject: | Re: BUG #17066: Cache lookup failed when null (unknown) is passed as anycompatiblemultirange |
| Date: | 2021-07-20 21:54:55 |
| Message-ID: | 640279.1626818095@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
Alexander Korotkov <aekorotkov(at)gmail(dot)com> writes:
> Do I understand correctly that enforce_generic_type_consistency() is
> called only after check_generic_type_consistency() returned true?
> If so, that means some of the checks are redundant. Therefore, we can
> replace ereport()'s with Assert()'s.
They are not redundant, IIRC. I forget the exact mechanism for
reaching them, but it likely has something to do with aggregates
or variadic functions.
In any case, apologies for taking so long to get back to this. Here's
a proposed patch (based in part on Neil's earlier patch).
regards, tom lane
| Attachment | Content-Type | Size |
|---|---|---|
| fix-bug-17066.patch | text/x-diff | 27.9 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | PG Bug reporting form | 2021-07-21 03:28:26 | BUG #17117: FailedAssertion at planner.c |
| Previous Message | Noah Misch | 2021-07-20 21:49:36 | Re: CREATE INDEX CONCURRENTLY does not index prepared xact's data |