| From: | Bruce Momjian <bruce(at)momjian(dot)us> |
|---|---|
| To: | Brendan Jurd <direvus(at)gmail(dot)com> |
| Cc: | pgsql-hackers(at)postgresql(dot)org, pgsql-patches(at)postgresql(dot)org, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Subject: | Re: [HACKERS] Add function for quote_qualified_identifier? |
| Date: | 2007-10-08 18:33:53 |
| Message-ID: | 200710081833.l98IXru16653@momjian.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers pgsql-patches |
Brendan Jurd wrote:
> On 9/29/07, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
> > I think we need more than one person's request to add this function.
>
> Well, I don't expect it would get requested. Most DBAs would likely
> look for the function in the docs, see it's not there and then just
> implement it themselves. Obviously it's not critical. But
> anticipating those little requirements and providing for them is one
> of the things that makes a piece of software a pleasure to use.
> "Batteries included" and all that.
I was just looking for someone else to say "Yea, I would like that too".
> Anyway, I seem to be flogging a horse which, if not dead, is surely
> mortally wounded. If quote_qualified_ident isn't desired, perhaps you
> can still use the regression test I included for quote_ident in the
> patch. The test is functional as a standalone item, and seems to fill
> a gap.
Well, we don't test everything and I don't remember problems in quoting
in the past, at least not enough to add another test.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://postgres.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2007-10-08 18:43:07 | Re: Getting to 8.3 beta1 |
| Previous Message | Tom Lane | 2007-10-08 17:34:29 | Re: [COMMITTERS] pgsql: Added the Skytools extended transaction ID module to contrib as |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Marko Kreen | 2007-10-08 19:47:14 | txid strtoull fix |
| Previous Message | Heikki Linnakangas | 2007-10-08 17:10:45 | Re: Including Snapshot Info with Indexes |