Martijn, * Martijn van Oosterhout (klep...@svana.org) wrote: > Note, my whole purpose for suggesting something like: > > include_auto_conf_file postgresql.auto.conf > > is because I want the file location to be configurable. If I put in my > configuration: > > include_auto_conf_file /etc/postgresql/9.4/postgresql.auto.conf
I'm not following the use-case here.
Why would it make sense for a file which is entirely managed by PG
automatically to be in /etc? Of course, by the same token you might ask
why it makes sense to let the user pick it at all, which holds some
merit- but I liked your idea because then an admin doesn't have to go
looking for the file but instead knows where it is. Perhaps comments in
the file stating where the auto.conf lives would be sufficient, but
those are too often nuked. :(
> it better put the options there. And if I comment the line out ALTER
> SYSTEM should stop working. If I put it at the beginning of the config
> then any other option in the file will override it (which we can detect
> and warn about). If I put it at the end of the file, ALTER SYSTEM
> overrides any statically configured options.
>
> And I can imagine hosting providers putting the configuration for
> memory usage, shared library directories and other such options after,
> and options like cpu_cost, enable_merge_join, etc before. That way
> they have fine grained control over which options the user can set and
> which not.
For my part, I'd honestly rather have the first things found be what's
picked and later things be ignored. If you want it controlled by ALTER
SYSTEM, then don't set it in postgresql.conf. The problem with that is
there's no "bootstrap" mechanism without actually modifying such configs
or making the configs be in auto.conf to begin with, neither of which is
ideal, imv.
I really hate the idea that someone could configure 'X' in
postgresql.conf and because the auto.conf line is later in the file,
it's able to override the original setting. Does not strike me as
intuitive at all.
Thanks,
Stephen
signature.asc
Description: Digital signature
