Re: PATCH: Unlogged tables re-initialization tests

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: David Steele <david(at)pgmasters(dot)net>
Cc: Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
Subject: Re: PATCH: Unlogged tables re-initialization tests
Date: 2018-03-02 04:59:47
Message-ID: 20180302045947.GG2259@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Mar 01, 2018 at 01:12:19PM -0500, David Steele wrote:
> But your point is well-taken. No symlinks are used in this test so it
> *should* work.
>
> Michael, what do you think?

Perl's symlink() does not work on Windows. It does not fly higher than
that, and that's the reason why a good chunk of the tests are skipped
for pg_basebackup. If perl was to have a version of symlink() which
works, say with junction points, or if Windows was to have a sane
symlink implementation (or with [1]?), or if it was possible to create
junction points using an in-core implementation in perl, then those
tests could not be skipped. But it seems that none of those scenarios
have happened yet.

From what I read in your patch, it seems to me that this test should
work. If they don't for whatever reason, your patch then does not give
a correct justification explaining why they should be skipped.

[1]: https://blogs.windows.com/buildingapps/2016/12/02/symlinks-windows-10/
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Yoshimi Ichiyanagi 2018-03-02 05:01:24 Re: [HACKERS][PATCH] Applying PMDK to WAL operations for persistent memory
Previous Message Amit Langote 2018-03-02 04:51:48 Re: 2018-03 Commitfest Summary (Andres #3)