On Mon, 18 May 2026 at 12:49, Álvaro Herrera <[email protected]> wrote: > > On 2026-May-18, Thom Brown wrote: > > > I saw this just got committed. Is there a reason the option handling > > for this, VACUUM, and EXPLAIN aren't done in one place? > > I don't think they take exactly the same options; but if you see room > for improvement, by all means send a patch.
I wasn't thinking of a shared handler for the options themselves, because, as you say, those differ per command. I meant the handling of the DefElem list: last-duplicate-wins, rejecting unknown options, coercion via defGetBoolean, etc. That part is the same everywhere, but each command writes its own set of checks, and the bug fixed here is what you get from that: a loop that enabled the option if any occurrence was ON, instead of just overwriting. So each command would hand its DefElem list to a common routine along with a table of valid option names and types. The routine iterates the list, rejects unknown names, coerces values, and applies last-wins for repeated options. Command-specific semantic checks would stay in the command. I could try writing a patch, but at this point in the dev cycle, it's probably something for later. Thom
