Decibel! <[EMAIL PROTECTED]> writes: > On Jul 15, 2008, at 4:58 PM, Tom Lane wrote: >> There are two ways we >> could do it: a bool column that is TRUE if the function is variadic, >> or an oid column that is the variadic array's element type, or zero >> if the function isn't variadic. The second would take more space but >> would avoid having to do a catalog lookup to get the element type in >> the case that the function is indeed variadic. I'm leaning to the >> second way but wanted to know if anyone objected?
> If you go the second route, I'd vote for it being NULL if the > function isn't variadic, unless that would play hell with the C side > of the catalog code... Getting rid of the check for null is *exactly* the point here --- AFAICT that's what's eating all the time in the existing code. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers