Re: Patches applied; initdb time!

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Thomas Lockhart <thomas(at)fourpalms(dot)org>
Cc: PostgreSQL Hackers List <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Patches applied; initdb time!
Date: 2002-04-21 21:22:47
Message-ID: 26761.1019424167@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

Thomas Lockhart <thomas(at)fourpalms(dot)org> writes:
> More specifically, the *only* compiler warning I see (other than the
> usual yacc/lex symbol warnings) is that a routine in gram.y,
> set_name_needs_quotes(), is defined but not used. Don't know where that
> routine came from, and afaik I didn't accidentally remove a reference
> when trying to merge changes...

Yeah, you did. However the routine could possibly go away now.
It was a hack I put in recently to handle cases like

regression=# create schema "MySchema";
CREATE
regression=# create schema "MyOtherSchema";
CREATE
regression=# set search_path TO "MySchema", "MyOtherSchema";
ERROR: SET takes only one argument for this parameter

Formerly gram.y merged the list items into a single string, and so it
needed to double-quote mixed-case names to prevent case folding when
the string got re-parsed later.

This example worked last week, and probably would work again if the
system were applying your new list-argument logic for search_path ...
but I'm not sure where to look to learn about that.

regards, tom lane

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Thomas Lockhart 2002-04-21 21:24:57 Re: Patches applied; initdb time!
Previous Message Joe Conway 2002-04-21 21:14:52 Re: Patches applied; initdb time!

Browse pgsql-hackers by date

  From Date Subject
Next Message Thomas Lockhart 2002-04-21 21:24:57 Re: Patches applied; initdb time!
Previous Message Joe Conway 2002-04-21 21:14:52 Re: Patches applied; initdb time!