Re: [PATCHES] Add function for quote_qualified_identifier? - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: [PATCHES] Add function for quote_qualified_identifier?
Date
Msg-id 200710081833.l98IXru16653@momjian.us
Whole thread Raw
In response to Re: [PATCHES] Add function for quote_qualified_identifier?  ("Brendan Jurd" <direvus@gmail.com>)
Responses Re: [PATCHES] Add function for quote_qualified_identifier?  (Alvaro Herrera <alvherre@commandprompt.com>)
List pgsql-hackers
Brendan Jurd wrote:
> On 9/29/07, Bruce Momjian <bruce@momjian.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@momjian.us>        http://momjian.us
  EnterpriseDB                             http://postgres.enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [COMMITTERS] pgsql: Added the Skytools extended transaction ID module to contrib as
Next
From: Bruce Momjian
Date:
Subject: Re: Getting to 8.3 beta1