At 2020-10-20 10:53:04 -0700, and...@anarazel.de wrote: > > > postgres=# ALTER SYSTEM SET max_worker_processes += 4; > > ALTER SYSTEM > > Much less clear that this is a good idea...
I agree it's less clear. I still think it might be useful in some cases (such as the example with max_worker_processes quoted above), but it's not as compelling as altering search_path/shared_preload_libraries. (That's partly why I posted it as a separate patch.) > It seems to me that appending and incrementing using the same syntax > is a) confusing b) will be a limitation before long. I understand (a), but what sort of limitation do you foresee in (b)? Do you think both features should be implemented, but with a different syntax, or are you saying incrementing should not be implemented now? > > These patches do not affect configuration file parsing in any way: > > its use is limited to "SET" and "ALTER xxx SET". > > Are you including user / database settings as part of ALTER ... SET? > Or just SYSTEM? Yes, it works the same for all of the ALTER … SET variants, including users and databases. -- Abhijit
