|From:||Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>|
|To:||Peter Eisentraut <peter_e(at)gmx(dot)net>|
|Subject:||Re: remove wal_level archive|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
Peter Eisentraut wrote:
> So we've had several rounds of discussions about simplifying replication
> configuration in general and the wal_level setting in particular. 
> Let's get something going.
I looked at this patch, which I think has got enough consensus that you
should just push forward with the proposed design -- in particular, just
remove one of archive or hot_standby values, not keep it as a synonym of
the other. If we're counting votes, I prefer keeping hot_standby over
The patch is nicely compact and applies, with only some fuzz.
I agree with changing all parts that say "XYZ or higher" to enumerate
the possible values.
It may be a good idea to have a look at Michael Paquier's recovery test
framework ( also in this commitfest: https://commitfest.postgresql.org/8/438/ )
and see how that is affected by this patch. Maybe the tests can find a
problem in this patch, and so perhaps you'd like to commit the tests
first, then this change on top.
I'm marking this as Ready for Committer, and setting you up as such for
this patch. If you would prefer not to commit, let me know and I can do
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
|Next Message||Pavel Stehule||2016-01-04 16:04:50||Re: custom function for converting human readable sizes to bytes|
|Previous Message||Tom Lane||2016-01-04 15:59:15||Re: Some 9.5beta2 backend processes not terminating properly?|