From: | Joe Conway <mail(at)joeconway(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-patches(at)postgresql(dot)org, Masaru Sugawara <rk73(at)sea(dot)plala(dot)or(dot)jp> |
Subject: | Re: quote_ident and schemas (was Re: [HACKERS] connectby with schema) |
Date: | 2002-11-22 23:21:48 |
Message-ID: | 3DDEBC0C.3030800@joeconway.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Tom Lane wrote:
> Since connectby takes a string parameter (correct?) for the table name,
> my advice would be to have it not do quote_ident, but instead expect the
> user to include double quotes in the string value if dealing with
> mixed-case names. Compare the behavior of nextval() for example:
>
> regression=# select nextval('Foo.Bar');
> ERROR: Namespace "foo" does not exist
> regression=# select nextval('"Foo"."Bar"');
> ERROR: Namespace "Foo" does not exist
> regression=# select nextval('"Foo.Bar"');
> ERROR: Relation "Foo.Bar" does not exist
>
OK. Attached patch removes calls within the function to quote_ident, requiring
the user to appropriately quote their own identifiers. I also tweaked the
regression test to deal with "value" becoming a reserved word.
If it's not too late, I'd like this to get into 7.3, but in any case, please
apply to HEAD.
Thanks,
Joe
p.s. There are similar issues in dblink, but they appear a bit more difficult
to address. I'll attempt to get them resloved this weekend, again in hopes to
get them applied before 7.3 is released.
Attachment | Content-Type | Size |
---|---|---|
tablefunc-fix.2002.11.22.1.patch | text/plain | 17.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2002-11-23 01:55:22 | Re: quote_ident and schemas (was Re: [HACKERS] connectby with schema) |
Previous Message | Tom Lane | 2002-11-22 23:05:35 | Re: "value" a reserved word |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2002-11-23 01:55:22 | Re: quote_ident and schemas (was Re: [HACKERS] connectby with schema) |
Previous Message | Neil Conway | 2002-11-22 20:13:05 | release note fixes (7.3 & HEAD) |