Andres Freund <and...@anarazel.de> writes: >> offset_clause: >> @@ -15435,6 +15442,7 @@ reserved_keyword: >> | ONLY >> | OR >> | ORDER >> + | PERCENT >> | PLACING >> | PRIMARY >> | REFERENCES
> Are we really ok with adding a new reserved keyword 'PERCENT' for this? I'm not. It doesn't look like it ought to be necessary to reserve it, either, given that we don't have to reduce the production right there. (If that doesn't work right away, try getting rid of row_or_rows in favor of spelling out those alternatives in separate productions.) More generally: using an undocumented list as the data structure for select_limit's result was already a pretty bad idea, I think, and this patch certainly makes it totally unreadable. Probably ought to refactor that to use some sort of struct. regards, tom lane