Re: [PATCH] Rename pg_switch_xlog to pg_switch_wal

From: Vladimir Rusinov <vrusinov(at)google(dot)com>
To: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Jim Nasby <Jim(dot)Nasby(at)bluetreble(dot)com>, David Steele <david(at)pgmasters(dot)net>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, Stephen Frost <sfrost(at)snowman(dot)net>, Cynthia Shang <cynthia(dot)shang(at)crunchydata(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH] Rename pg_switch_xlog to pg_switch_wal
Date: 2017-01-03 13:26:50
Message-ID: CAE1wr-zZifGZEPwYLKJjo-yCuZQwHAa54nF8DoT7MU0Un_Buwg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Jan 3, 2017 at 11:56 AM, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
wrote:

> Yeah, let's make the life of users just easier if we can, without any
> extension. Some people are likely going to forget to enable it anyway,
> and some more don't like installing the package dedicated to contrib
> modules.
>

I think I +1 on this.
I've did a github search on these function names and there is a lot of code
that use them. E.g. there is 8.5k hits for pg_last_xlog_location
<https://github.com/search?q=pg_last_xlog_replay_location&type=Code&utf8=%E2%9C%93>;
a lot of them a forks and copy-pastes, but still, that's quite a lot. Let's
keep the aliases around for couple of versions after which hopefully a lot
of the code will be updated.

After I'm done with a patch I can take a look if there's a way to mass-fill
issues to affected projects on github (and maybe other hosting providers)
and ask them to migrate to new function names. This is of course will not
fix everything, but it would be a significant chunk. I suspect a lot of
private tools/scripts borrow snippets from github as well.

--
Vladimir Rusinov
Storage SRE, Google Ireland

Google Ireland Ltd.,Gordon House, Barrow Street, Dublin 4, Ireland
Registered in Dublin, Ireland
Registration Number: 368047

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message valeriof 2017-01-03 13:32:27 Re: Logical decoding - filtering tables
Previous Message Pavel Stehule 2017-01-03 13:14:27 Re: proposal: session server side variables