Re: s/xlog/wal/ in tools and function names? - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: s/xlog/wal/ in tools and function names?
Date
Msg-id CAB7nPqTrKZmxo3R4bSursnzN9zdNCC8TaNnuFE0xtAnWwBF32w@mail.gmail.com
Whole thread Raw
In response to Re: s/xlog/wal/ in tools and function names?  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: s/xlog/wal/ in tools and function names?  (Vladimir Rusinov <vrusinov@google.com>)
Re: s/xlog/wal/ in tools and function names?  (Euler Taveira <euler@timbira.com.br>)
List pgsql-hackers
On Fri, Dec 2, 2016 at 3:21 AM, Robert Haas <robertmhaas@gmail.com> wrote:
> On Thu, Dec 1, 2016 at 10:29 AM, Vladimir Rusinov <vrusinov@google.com> wrote:
>> I've found myself wondering "where is my xlog" after running
>> pg_switch_xlog() in 10.0.
>>
>> Renaming pg_xlog to pg_wal created inconsistency between tools, function
>> names and directory name on disk.
>>
>> Should we also:
>>
>> - rename pg_switch_xlog and friends to pg_switch_wal?
>> - rename pg_recievexlog to pg_revievewal (and others in bin/)?
>> - rename pg_xlogdump to pg_waldump?
>
> I think yes to all.

I was hesitant to propose that, but if there is a will do move
everything... Documentation would point to different pages if the
utilities are renamed, so that's not helpful when comparing features
across major releases... We may want to keep those files with their
historical names.

>> - if we do rename, should we keep aliases for functions and symlinks for
>> tools?
>
> I think no.

Better to do breakages in a single release rather than spreading them
across releases. While at it and because we are on a crazy trend, one
thing we could as well consider is removing pg_xlog_location_diff().
It has lost sense since pg_lsn has been introduced.

>> - anything else?
>
> There are some SQL-callable functions that should probably be renamed
> to match, too.

=# select proname from pg_proc where proname ~ 'xlog';            proname

---------------------------------pg_current_xlog_locationpg_current_xlog_insert_locationpg_current_xlog_flush_locationpg_xlogfile_name_offsetpg_xlogfile_namepg_xlog_location_diffpg_last_xlog_receive_locationpg_last_xlog_replay_locationpg_is_xlog_replay_pausedpg_switch_xlogpg_xlog_replay_pausepg_xlog_replay_resume
(12 rows)
-- 
Michael



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: PSQL commands: \quit_if, \quit_unless
Next
From: Haribabu Kommi
Date:
Subject: Re: btree_gin and btree_gist for enums