Hi, On 2019-05-15 11:05:31 -0400, Alvaro Herrera wrote: > After eyeballing the giant patch set you sent[1], I think EXEC is a > horrible keyword to use -- IMO it should either be the complete word > EXECUTE, or we should pick some other word. I realize that we do not > want to have different sets of keywords when using the legacy syntax (no > parens) vs. new-style (with parens), but maybe we should just not > support the EXECUTE keyword in the legacy syntax; there's already a > number of options we don't support in the legacy syntax (BUFFERS, > TIMING), so this isn't much of a stretch.
That seems too confusing. > [1] I think if you just leave out the GUC print from the changes, it > becomes a reasonable patch series. Yea, it really should be small incremental changes instead of proposals to "just change everything". Greetings, Andres Freund