- Notifications
You must be signed in to change notification settings - Fork5
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 changed| 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)