From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>, Alexey Kondratov <a(dot)kondratov(at)postgrespro(dot)ru>, Liudmila Mantrova <l(dot)mantrova(at)postgrespro(dot)ru>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, Andrey Borodin <x4mmm(at)yandex-team(dot)ru>, David Steele <david(at)pgmasters(dot)net>, Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Dmitry Dolgov <9erthalion6(at)gmail(dot)com>, vladimirlesk(at)yandex-team(dot)ru, dsarafan(at)yandex-team(dot)ru |
Subject: | Re: [Patch] pg_rewind: options to use restore_command from recovery.conf or command line |
Date: | 2020-01-19 21:08:37 |
Message-ID: | 20200119210837.GA1485@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2020-Jan-19, Michael Paquier wrote:
> So using WAL to tell about a WAL segment file is wrong, WALs is not a
> term that actually exists.
I agree.
> So, in my opinion, it is fine to use "WAL file", "WAL segment" or even
> "WAL segment file".
Agreed with these three terms -- "WAL file" seems to be the most common,
but the other two terms you suggest are just as good.
> +use File::Glob ':bsd_glob';
> +use File::Path qw(remove_tree make_path);
> +use File::Spec::Functions qw(catdir catfile);
> Is this compatible with our minimum perl requirements for the TAP
> tests?
By and large, we just join elements with a slash "foo/bar" to create
path names; no need for catdir or catfile, ISTM.
I *think* :bsd_glob should be available in all the Perl versions we
support, but I'm not sure that we really need it. We seem to do just
fine with regular glob elsewhere.
PostgresNode already uses File::Path's rmtree. Looking at its manual,
maybe the safest bet is to change all that to remove_tree.
Not sure about make_path.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2020-01-19 21:48:05 | Re: [Patch] pg_rewind: options to use restore_command from recovery.conf or command line |
Previous Message | Ranier Vilela | 2020-01-19 20:49:08 | [PATCH] Windows port, fix some resources leaks |