Re: [HACKERS] [PATCH] Rename pg_switch_xlog to pg_switch_wal - Mailing list pgsql-hackers

From Stephen Frost
Subject Re: [HACKERS] [PATCH] Rename pg_switch_xlog to pg_switch_wal
Date
Msg-id 20161229175928.GU18360@tamriel.snowman.net
Whole thread Raw
In response to Re: [HACKERS] [PATCH] Rename pg_switch_xlog to pg_switch_wal  (Cynthia Shang <cynthia.shang@crunchydata.com>)
List pgsql-hackers
Cynthia,

Please don't top-post on the PG mailing lists but rather write responses
in-line.

* Cynthia Shang (cynthia.shang@crunchydata.com) wrote:
> I have never heard of coding standards where naming conventions required a function/variable name match a directory
orfile name. It seems that would be restrictive.  

We aren't discussing a coding standard, we're talking about changing the
references from 'xlog' to 'wal' to match the change we did to the
directory, to be consistent.

> I'm not trying to pick a fight, I just think the pros should outweigh the cons when choosing a path forward. In this
caseI see lots of cons and one partial pro; partial because renaming only user facing functions and documentation will
createinconsistency within the Postgres code for all of the following. It sounds as if your minds are already made up,
whichsaddens me but I will say nothing further on the matter. 

All backwards incompatible changes are judgement calls and people are
certainly welcome to have different opinions.  I have a pretty strong
feeling about this particular change being worthwhile and also pretty
long overdue.

[... list of references to xlog in the code ...]

No need to include a list, I can find them myself. :)

Nearly all of the references listed are internal functions or references
in the code.  Those don't need to be changed, just the user-facing
pieces, which is quite a bit less.

The doxygen website is generated directly off of the code and isn't
user-facing documentation.

Thanks!

Stephen

pgsql-hackers by date:

Previous
From: Cynthia Shang
Date:
Subject: Re: [HACKERS] [PATCH] Rename pg_switch_xlog to pg_switch_wal
Next
From: Fabien COELHO
Date:
Subject: Re: [HACKERS] proposal: session server side variables