| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
| Cc: | samuel(dot)horwitz(at)gmail(dot)com, PostgreSQL mailing lists <pgsql-bugs(at)postgresql(dot)org> |
| Subject: | Re: BUG #14876: Segmentation fault with JSONB column used in store proc that gets used by view and later altered |
| Date: | 2017-10-28 01:09:23 |
| Message-ID: | 31242.1509152963@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
Michael Paquier <michael(dot)paquier(at)gmail(dot)com> writes:
> Sorry for coming up late in the game. I can see that you have pushed a
> patch as d5b760e, but back-paddled a bit on d76886c. After some
> analysis of things around, I think that you got it right. One comment
> I have first though is that you could have used forboth as there is no
> point to go through the target list entries once there are no more
> aliases. Or target list entries marked as resjunk do not have an
> expended reference name?
Right, there's no entry in the outer RTE for resjunk columns.
(In practice, resjunk entries are at the end of the tlist so that it
wouldn't really matter, but I try to keep code from assuming that.)
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | jan.przemyslaw.wojcik | 2017-10-28 01:10:36 | BUG #14878: pg_trgm - word_similarity() inconsistent with description. |
| Previous Message | Michael Paquier | 2017-10-28 00:37:29 | Re: BUG #14876: Segmentation fault with JSONB column used in store proc that gets used by view and later altered |