forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit3d48045
committed
> 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 workbecause 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 caughtthe bug in the first place.Thanks,Matthew T. O'Connor1 parent188eda0 commit3d48045
1 file changed
+8
-3
lines changedLines changed: 8 additions & 3 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
88 | 88 |
| |
89 | 89 |
| |
90 | 90 |
| |
91 |
| - | |
| 91 | + | |
92 | 92 |
| |
93 | 93 |
| |
94 | 94 |
| |
95 | 95 |
| |
96 | 96 |
| |
97 | 97 |
| |
98 |
| - | |
99 |
| - | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
100 | 104 |
| |
| 105 | + | |
101 | 106 |
| |
102 | 107 |
| |
103 | 108 |
| |
|
0 commit comments
Comments
(0)