From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
Cc: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Paul Ramsey <pramsey(at)cleverelephant(dot)ca>, Andres Freund <andres(at)anarazel(dot)de>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Simon Riggs <simon(at)2ndquadrant(dot)com>, Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org>, Robert Haas <robertmhaas(at)gmail(dot)com> |
Subject: | Re: [PATCH] postgres_fdw extension support |
Date: | 2015-08-21 16:03:38 |
Message-ID: | 20150821160338.GA25495@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Aug 21, 2015 at 12:55:39PM -0300, Alvaro Herrera wrote:
> Michael Paquier wrote:
>
> > if (needlabel)
> > appendStringInfo(buf, "::%s",
> > -
> > format_type_with_typemod(node->consttype,
> > -
> > node->consttypmod));
> > +
> > format_type_be_qualified(node->consttype));
> > Pondering more about this one, I think that we are going to need a
> > new routine in format_type.c to be able to call
> > format_type_internal as format_type_internal(type_oid, typemod,
> > true/false, false, true). If typemod is -1, then typemod_given
> > (the third argument) is false, otherwise typemod_given is true.
> > That's close to what the C function format_type at the SQL level
> > can do except that we want it to be qualified. Regression tests
> > will need an update as well.
>
> I don't know what's going on here, but please look at the patch
> posted by Alexander Shulgin in the thread about JSON DDL deparse
> today; there's some additional stuff in format_type.c there that is
> probably useful to share between these two patches.
Should that stuff be its own stand-alone patch?
Cheers,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate
From | Date | Subject | |
---|---|---|---|
Next Message | Piotr Stefaniak | 2015-08-21 16:32:40 | A few cases of left shifting negative integers |
Previous Message | Alvaro Herrera | 2015-08-21 15:55:39 | Re: [PATCH] postgres_fdw extension support |