From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com> |
Cc: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: proposal: doc: simplify examples of dynamic SQL |
Date: | 2015-03-20 20:28:38 |
Message-ID: | 20150320202838.GM6317@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Mar 20, 2015 at 08:43:21AM -0700, David G. Johnston wrote:
> On Thu, Mar 19, 2015 at 6:49 PM, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
> It is making a point about nulls and stuff. There are later queries
> that use format().
>
> I thought maybe you meant those but your specific mention of "
>
> There are other places later in the docs" confused me since you made changes
> before and after that specific section.
>
> Those examples need to be somewhere and it doesn't seem like a undesireable
> enough setup that major reconstructive surgery is warranted to try and move
> them elsewhere.
Yes, agreed.
> I was actually referring to chapter 9
>
> http://www.postgresql.org/docs/9.4/interactive/functions-string.html
>
> The table definitions of the quote_* function should have a comment about
> their equivalency to format %I and %L
I think it is going to be awkward to mention a much more complex
function, format(), when covering a simle quote function.
> Also, in 9.4.1 (format -> type) would be the most obvious place for the
> equivalency of the format %I and %L to quote_*
Yes, added.
Update patch attached and URL udpated with current patch too.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ Everyone has their own god. +
Attachment | Content-Type | Size |
---|---|---|
format.diff | text/x-diff | 7.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2015-03-20 20:33:59 | Re: proposal: doc: simplify examples of dynamic SQL |
Previous Message | Stephen Frost | 2015-03-20 19:49:50 | Re: Remove fsync ON/OFF as a visible option? |