Re: Adding an example for replication configuration to pg_hba.conf
Magnus Hagander <magnus@hagander.net>
From: Magnus Hagander <magnus@hagander.net>
To: Dave Page <dpage@pgadmin.org>
Cc: Selena Deckelmann <selena@chesnok.com>, Alvaro Herrera <alvherre@commandprompt.com>, Greg Smith <greg@2ndquadrant.com>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2011-05-19T18:05:54Z
Lists: pgsql-hackers
On Thu, May 19, 2011 at 11:09, Dave Page <dpage@pgadmin.org> wrote: > On Thu, May 19, 2011 at 2:44 PM, Selena Deckelmann <selena@chesnok.com> wrote: >> On Wed, May 18, 2011 at 8:20 PM, Alvaro Herrera >> <alvherre@commandprompt.com> wrote: >>> Excerpts from Greg Smith's message of mié may 18 23:07:13 -0400 2011: >>>> Two things that could be changed from this example to make it more useful: >>> >>>> -The default database is based on your user name, which is postgres in >>>> most packaged builds but not if you compile your own. I don't know >>>> whether it's practical to consider substituting that into this file, or >>>> if it's just enough to mention that as an additional doc comment. >>> >>> You mean the default username, not the default database, but yeah; so do >>> we need a @default_username@ token to be replaced by initdb with >>> whatever it has as effective_user? (In this case the patch is no longer >>> 2 lines, but still should be trivial enough). >> >> That would be nice. So, we just add that token to initdb? Seems simple. >> >> I added some explanation of the all vs replication bit in the header comments. >> >> Revision attached. > > Looks good to me. > > As I mentioned offlist, I'd like it in teal please. Applied with some further minor bikeshedding (remove trailing spaces, rewrap so columns aren't wider than 80 chars, etc) -- Magnus Hagander Me: http://www.hagander.net/ Work: http://www.redpill-linpro.com/