Hi, I worked on unifying how the utility commands (VACUUM, ANALYZE, EXPLAIN, CHECKPOINT, REPACK) handle duplicate options. It came up in one of my patches here [1].
Based on that, I'm leaning towards last-wins rather than erroring on duplicate options. It matches what REPACK/VACUUM/EXPLAIN already do, and REPACK was recently fixed in-tree to be last-wins. But let me know your feedback, since COPY and WAIT FOR follow the error mechanism: 1/ is it better to have the error everywhere instead? Or 2/ go with the attached rough draft of the patch, which adds a centralized helper that collapses duplicates to last-wins and has each command called once? [1] https://www.postgresql.org/message-id/CAA-aLv57ymRoDJ%2B2UnHCE7WDB_FH5pW9%2BbFX7z5wEegudPrdRQ%40mail.gmail.com Regards, Baji Shaik
0001-Centralize-last-wins-handling-of-duplicate-utility-c.patch
Description: Binary data
