| From: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Michael Paquier <michael(at)paquier(dot)xyz> |
| Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Robert Haas <robertmhaas(at)gmail(dot)com> |
| Subject: | Re: prokind column (was Re: [HACKERS] SQL procedures) |
| Date: | 2018-02-28 22:37:11 |
| Message-ID: | 3b446110-24ce-3e45-ad81-826b933cdbb0@2ndquadrant.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 2/28/18 15:45, Tom Lane wrote:
> I have reviewed this patch and attach an updated version below.
> I've rebased it up to today, fixed a few minor errors, and adopted
> most of Michael's suggestions. Also, since I remain desperately
> unhappy with putting zeroes into prorettype, I changed it to not
> do that ;-) ... now procedures have VOIDOID as their prorettype,
> and it will be substantially less painful to allow them to return
> some other scalar result in future, should we wish to. I believe
> I've found all the places that were relying on prorettype == 0 as
> a substitute for prokind == 'p'.
I have just posted "INOUT parameters in procedures", which contains some
of those same changes. So I think we're on the same page. I will work
on consolidating this.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andrew Dunstan | 2018-02-28 22:49:43 | Re: ALTER TABLE ADD COLUMN fast default |
| Previous Message | Peter Eisentraut | 2018-02-28 22:28:51 | INOUT parameters in procedures |