> On 27 Jan 2018, at 00:36, Tom Lane <t...@sss.pgh.pa.us> wrote: > > I've pushed this mostly as-is.
Thanks! > I also took out the parser changes related to > allowing unquoted PARALLEL in old-style CREATE AGGREGATE, because that > is not a goal I consider worthy of adding extra grammar complexity. > We don't document that PARALLEL works there, and there has never been > any expectation that deprecated legacy syntax would grow new options > as needed to have feature parity with the modern syntax. Ok, didn’t know old syntax wasn’t extended to support new options so I went after it having run into the regress tests using it. > I also trimmed the new regression test cases a bit, as most of them seemed > pretty duplicative. Makes sense, they were made verbose to aid review but were too chatty for inclusion. cheers ./daniel
