| From: | "Matthew T(dot) O'Connor" <matthew(at)zeut(dot)net> |
|---|---|
| To: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
| Cc: | Adam Kavan <akavan(at)cox(dot)net>, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: Another small bug (pg_autovacuum) |
| Date: | 2003-09-11 04:03:44 |
| Message-ID: | 1063253023.28079.3.camel@zeutrh9 |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, 2003-09-10 at 15:57, Bruce Momjian wrote:
> I assume the attached patch is what you want done to fix this. Applied.
>
> It quotes table names for vacuum and analyze, and uppercases the
> keywords for clarity.
Yeah, this is basically what I meant, sorry I didn't get to it quicker.
However, I tested it out a little and the patch you made doesn't work
because it produces commands like:
VACUUM ANALYZE "public.FooBar"
Which doesn't work, so I made my own patch that creates commands like:
VACUUM ANALYZE "public"."FooBar"
This allows for mixed case schema names as well as tables.
Adam, can you please give this a test as you are the person who caught
the bug in the first place.
Thanks,
Matthew T. O'Connor
| Attachment | Content-Type | Size |
|---|---|---|
| diff | text/x-patch | 3.5 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jeroen Ruigrok/asmodai | 2003-09-11 05:35:13 | Re: constraint modification on todo list |
| Previous Message | Tom Lane | 2003-09-11 03:14:46 | Re: massive quotes? |