Re: [PATCH] Rename pg_switch_xlog to pg_switch_wal
Stephen Frost <sfrost@snowman.net>
From: Stephen Frost <sfrost@snowman.net>
To: Andres Freund <andres@anarazel.de>
Cc: Michael Paquier <michael.paquier@gmail.com>, Peter Eisentraut <peter.eisentraut@2ndquadrant.com>, Robert Haas <robertmhaas@gmail.com>, Fujii Masao <masao.fujii@gmail.com>, Vladimir Rusinov <vrusinov@google.com>, Tom Lane <tgl@sss.pgh.pa.us>, Euler Taveira <euler@timbira.com.br>, David Steele <david@pgmasters.net>, Jim Nasby <Jim.Nasby@bluetreble.com>, Cynthia Shang <cynthia.shang@crunchydata.com>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2017-01-30T04:44:09Z
Lists: pgsql-hackers
* Andres Freund (andres@anarazel.de) wrote: > On 2017-01-28 08:47:03 +0900, Michael Paquier wrote: > > On Sat, Jan 28, 2017 at 8:03 AM, Peter Eisentraut > > <peter.eisentraut@2ndquadrant.com> wrote: > > > On 1/26/17 2:05 PM, Robert Haas wrote: > > >> I do not think it can be right to rename the directory and not > > >> anything else. > > > > > > I think this is the root of the confusion. > > > > > > A lot of people apparently consented to renaming pg_xlog with the > > > understanding that that's it, whereas other people understood it as > > > consensus for renaming everything. > > > > > > You really have (at least) three options here: > > > > > > 1. Rename nothing > > > 2. Rename directory only > > > 3. Rename everything > > > > > > and you need to count the votes of each pair separately. > > > > 4. Rename everything with aliases. > > > > I would vote for 4., or 3. for consistency if 4. is not an eligible choice. > > 2, 4, 1, 3 For me, it's: 3, 4, 1, 2. I tend to agree with the other discussion, which points out that aliases for everything isn't really practical.. Of course, I'd also like to drop things like pg_user and pg_shadow one day... Never thought they'd last anywhere near this long when I first wrote those views, 12 years ago... Thanks! Stephen
Commits
-
Remove all references to "xlog" from SQL-callable functions in pg_proc.
- 806091c96f9b 10.0 landed
-
Replace pg_shadow and pg_group by new role-capable catalogs pg_authid
- 7762619e9527 8.1.0 cited