2018-01-02 17:47 GMT+01:00 Tom Lane <t...@sss.pgh.pa.us>: > Robert Haas <robertmh...@gmail.com> writes: > > I agree that we need this, but using prorettype = InvalidOid to do it > > might not be the best way, because it only works for procedures that > > don't return anything. If a procedure could return, say, an integer, > > Good point, because that is possible in some other systems, and so > somebody is going to ask for it at some point. > > > Anyway, I think it would be better to invent an explicit way to > > represent whether something is a procedure rather than relying on > > overloading prorettype to tell us. > > +1 --- seems like a new bool column is the thing. Or may we should merge > "proisprocedure" with proisagg and proiswindow into an enum prokind? > Although that would break some existing client-side code. >
+1 Pavel > PS: I still strongly disagree with allowing prorettype to be zero. > > regards, tom lane > >
